-
Notifications
You must be signed in to change notification settings - Fork 16.4k
Some minor updates to the Airflow release Doc #31024
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
Conversation
fb5fb14 to
29ad965
Compare
| check=True, | ||
| ) | ||
| console_print("Files pushed to svn") | ||
| os.chdir(repo_root) |
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.
I don't think you should need this change. remove_old_releases moves to a different dir anyways, then moves back to the repo root.
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.
At that point, we are in the "versions" directory. We can't change to "asf-dist..." right there because it cannot be found.
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.
I see. Can we the change remove_old_releases to do this "reset to repo root" instead?
| check=True, | ||
| ) | ||
| console_print("Files pushed to svn") | ||
| os.chdir(repo_root) |
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.
I see. Can we the change remove_old_releases to do this "reset to repo root" instead?
bfb5e5c to
898c9b2
Compare
Update to what I noticed during the 2.6.0 release