-
Notifications
You must be signed in to change notification settings - Fork 260
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
Add release instructions for Release Candidate #809
Merged
Merged
Conversation
This file contains 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
blrchen
requested review from
xiaoyongzhu,
mufajjul,
jainr,
edwincheung and
Yuqing-cat
November 1, 2022 10:10
xiaoyongzhu
previously approved these changes
Nov 1, 2022
Yuqing-cat
previously approved these changes
Nov 1, 2022
Yuqing-cat
approved these changes
Nov 1, 2022
mufajjul
approved these changes
Nov 1, 2022
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.
All looks good.
loomlike
pushed a commit
to loomlike/feathr
that referenced
this pull request
Nov 1, 2022
* Add release instructions for Release Candidate * Add a section for release versioning * Add a section for overall process triggered by the release manager
xiaoyongzhu
pushed a commit
that referenced
this pull request
Nov 23, 2022
* Fix xdist test error. Also make a small cleanup some codes Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> * Revert "Revert 756 (#798)" This reverts commit ff438f5. * revert 798 (revert756 - example notebook refactor). Also add job_utils unit tests Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> * Update test_azure_spark_e2e.py * Fix doc dead links (#805) This PR fixes dead links detected in latest ci run. The doc scan ci action has been updated to run on main only, as running this in PR frequently reports false alarm due to changes in CI not deployed. * Improve UI experience and clean up ui code warnings (#801) * Add DataSourcesSelect and FlowGraph and ResizeTable components. Fix all warning and lint issues. Signed-off-by: Boli Guan <ifendoe@gmail.com> * Add CardDescriptions component and fix ESlint warning. Signed-off-by: Boli Guan <ifendoe@gmail.com> * Update FeatureDetails page title. Signed-off-by: Boli Guan <ifendoe@gmail.com> * Rename ProjectSelect Signed-off-by: Boli Guan <ifendoe@gmail.com> Signed-off-by: Boli Guan <ifendoe@gmail.com> * Add release instructions for Release Candidate (#809) * Add release instructions for Release Candidate * Add a section for release versioning * Add a section for overall process triggered by the release manager * Bump version to 0.9.0-rc1 (#810) * Fix tests to use mocks and fix get_result_df's databricks behavior Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> * fix tem file to dir Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> * checkout the feature_derivations.py from main (it was temporally changed to goaround previous issues) Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> * Remove old databricks sample notebook. Change pip install feathr from the github main branch to pickup the latest changes always Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> * Fix config and get_result_df for synapse * Fix generate_config to accept all the feathr env var config name Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> * Add more pytests Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> * Use None as default dataformat in the job_utils. Instead, set 'avro' as a default output format to the job tags from the client Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> * Change feathr client to mocked object Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> * Change timeout to 1000s in the notebook Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com> Signed-off-by: Boli Guan <ifendoe@gmail.com> Co-authored-by: Blair Chen <blrchen@hotmail.com> Co-authored-by: Blair Chen <blrchen@users.noreply.github.com> Co-authored-by: Boli Guan <ifendoe@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
This PR adds release instructions for Release Candidate. The updated doc can be previewed at https://github.com/feathr-ai/feathr/blob/blair/rc-release-doc/docs/dev_guide/feathr_overall_release_guide.md
How was this PR tested?
Does this PR introduce any user-facing changes?