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

feat(ci/cd): set version to 0.dev #899

Merged
merged 2 commits into from
Aug 29, 2024
Merged

feat(ci/cd): set version to 0.dev #899

merged 2 commits into from
Aug 29, 2024

Conversation

esolitos
Copy link
Contributor

@esolitos esolitos commented Aug 29, 2024

What

  • Update versions in the repo to 0.dev
  • Remove the "Create PR" step during. the release process
  • Update badges in the README file
  • Re-enable the workflows to publish the package when a release is created

Why

Before this change each time we would release a new version we would also need to update the version in this repo (examples here and here, ...), this is s superfluous step which was required mostly to ensure that the CI workflows would work when doing pip install -e . as this requires a valid version.

After some discussion with @thomasht86 we reached the conclusion that we can set the ".dev" version (where X is the current major version) in those files and we update the version in those files during the release process for pypi.

This commit updates the pyvespa version to 0.dev in the pyproject.toml and vespa/__init__.py files. The changes are made to ensure the project's version is accurately reflected across the codebase.
@esolitos esolitos changed the title feat: update version to 0.dev feat: set version to 0.dev Aug 29, 2024
Copy link
Collaborator

@thomasht86 thomasht86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks smooth!
Should we uncomment "on release" for this workflow, so that we actually test and use this workflow on next release?

@esolitos esolitos self-assigned this Aug 29, 2024
This commit enables automatic PyPi release for the 'pyvespa' library when a new release is published on GitHub. Previously, the release workflow was disabled, but this change now triggers the release process when a new release is published.

The key changes are:
- Modify the release trigger to listen for 'published' events instead of 'workflow_dispatch'
- Add an environment variable 'RELEASE_REF' to ensure the version is available, even if the 'GITHUB_REF' variable is not set correctly
- Update the version extraction logic to use the 'RELEASE_REF' variable instead of 'GITHUB_REF'
@esolitos esolitos marked this pull request as ready for review August 29, 2024 10:44
@esolitos esolitos requested a review from thomasht86 August 29, 2024 10:52
@esolitos esolitos changed the title feat: set version to 0.dev feat(ci/cd): set version to 0.dev Aug 29, 2024
Copy link
Collaborator

@thomasht86 thomasht86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice 👏
Let us test next week.

@thomasht86 thomasht86 merged commit b78629b into master Aug 29, 2024
31 of 42 checks passed
@thomasht86 thomasht86 deleted the exclude-version-prs branch August 29, 2024 11:10
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