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

Update release guide #167

Merged
merged 7 commits into from
Aug 30, 2024
Merged

Update release guide #167

merged 7 commits into from
Aug 30, 2024

Conversation

yundai424
Copy link
Collaborator

Summary

conclude some learnings from this release :D

Testing Done

  • Hardware Type:
  • run make test to ensure correctness
  • run make checkstyle to ensure code style
  • run make test-convergence to ensure convergence

lancerts
lancerts previously approved these changes Aug 29, 2024
Copy link
Collaborator

@ByronHsu ByronHsu left a comment

Choose a reason for hiding this comment

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

Please also add how to autogen changelogs

CONTRIBUTING.md Outdated
3. Create a new release based on the current HEAD
4. New pip uploading will be triggered upon a new release
3. Create a new release based on the current HEAD, tag name using `v<version number>` for example `v0.2.0`. Alternatively, If you want to create release based on a different commit hash, `git tag v0.2.0 <commit hash> && git push origin v0.2.0`, and create release based on this tag
4. New pip uploading will be triggered upon a new release. NOTE: Both pre-release and official release will trigger the workflow to build wheel and publish to pypi, so please be sure that step 1-3 are followed correctly!
Copy link
Collaborator

Choose a reason for hiding this comment

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

so say if i pre-release one, then official release will conflict with the pre one because it already exists?

Copy link
Collaborator Author

@yundai424 yundai424 Aug 29, 2024

Choose a reason for hiding this comment

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

yup as soon as pre-release is out, the build & upload workflow will be triggered. When converting from pre-release to official release, no workflow will be fired. Feeling it would be nice if we could modify the workflow to make it run only for official release 🤔

@lancerts lancerts requested a review from ByronHsu August 30, 2024 00:59
@ByronHsu ByronHsu merged commit ff24de8 into main Aug 30, 2024
2 checks passed
@ByronHsu ByronHsu deleted the yundai424-patch-1 branch August 30, 2024 23:49
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.

3 participants