Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix version adjusting logic for snapshots #190

Merged
merged 1 commit into from
Apr 11, 2024
Merged

Conversation

ericanderson
Copy link
Member

@ericanderson ericanderson commented Apr 11, 2024

Version bumps with snapshots have -whatever in them which breaks the semver logic. We are instead just assuming its a release version to get through it.

This also fixes a bug in the logic for auto version updating.

This change also pushes the version expected for generated code into one central place per generated sdk which should prevent future repo churn as we rev client versions.

@ericanderson ericanderson merged commit f7287ae into main Apr 11, 2024
5 checks passed
@ericanderson ericanderson deleted the ea.fix.builds branch April 11, 2024 21:13
ericanderson added a commit that referenced this pull request Apr 19, 2024
* origin/main:
  Ea.release script (#203)
  Add pivot to for 1.1 (#158)
  Rename bulk actions to batch actions (#163)
  Make sure we publish with tags for snapshots (#194)
  Fix version adjusting logic for snapshots (#190)
  Maybe fix flaky test (#191)
  Make sure we catch pnpm build failures before merge (#189)
  Change release process and automate version updates (#188)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants