-
Notifications
You must be signed in to change notification settings - Fork 1.3k
chore: Update docs with new release workflow #2898
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
ecf99f0
Update docs
kevjumba 54d9fbe
Fix
kevjumba 00651be
Fix
kevjumba 748213b
Fix
kevjumba 136258d
Fix
kevjumba b779e51
Fix
kevjumba 1820ae9
Fix
kevjumba cdb5005
Fix
kevjumba 4f1c410
Fix
kevjumba ae4bc72
Fix
kevjumba 256bc63
Fix
kevjumba a8ca125
Fix
kevjumba 8bc5b1c
Fix
kevjumba bb2e4c7
Fix
kevjumba File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Fix
Signed-off-by: Kevin Zhang <kzhang@tecton.ai>
- Loading branch information
commit 256bc63635c2b4a97ca50d29e4fe3111f787b715
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -4,8 +4,7 @@ | |
|
|
||
| For Feast maintainers, these are the concrete steps for making a new release. | ||
|
|
||
| ### Pre-release Verification (Verification that wheels are built correctly) | ||
|
|
||
| ### Pre-release Verification (Verification that wheels are built correctly) for minor release. | ||
| 1. Merge upstream master changes into your **fork**. Make sure you are running the workflow off of your fork! | ||
| 2. Create a tag manually for the release on your fork. For example, if you are doing a release for version 0.22.0, create a tag by doing the following. | ||
| - Checkout master branch and run `git tag v0.22.0`. | ||
|
|
@@ -14,6 +13,13 @@ For Feast maintainers, these are the concrete steps for making a new release. | |
| 4. Look for the header `This workflow has a workflow_dispatch event trigger` and click `Run Workflow` on the right. | ||
| 5. Run the workflow off of the tag you just created(`v0.22.0` in this case) and verify that the workflow worked. | ||
|
|
||
| ### Pre-release Verification (Verification that wheels are built correctly) for patch release. | ||
| 1. Check out the branch of your release (e.g `v0.22-branch` on your local fork) and push this to your fork. | ||
| 2. Cherry pick commits that are relevant to the patch release onto your forked branch. | ||
| 3. Add a patch release tag (e.g `v0.22.1`) by running `git tag <tag>`. | ||
| 4. Push tags to origin branch with `git push origin <tag>`. | ||
| 5. Kick off `build_wheels` workflow in the same way as is detailed in the last section off of the patch release tag. | ||
|
|
||
| ### Release | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What stuff does this workflow do? Python release? Java release? UI release? Helm charts? Docker images?
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Clarified |
||
| 1. Generate a [Personal Access Token](https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/creating-a-personal-access-token) or retrieve your saved personal access token. | ||
| - The personal access token should have all of the permissions under the `repo` checkbox. | ||
|
|
||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What kind of verification?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clarified.