Skip to content

Add cherry-pick notes to the release README#17610

Merged
kaxil merged 3 commits into
apache:mainfrom
astronomer:release-readme
Aug 13, 2021
Merged

Add cherry-pick notes to the release README#17610
kaxil merged 3 commits into
apache:mainfrom
astronomer:release-readme

Conversation

@andrewgodwin

Copy link
Copy Markdown
Contributor

This adds some more detail to the release README about the process of selecting what is in a release, including the fact that the release manager has some discretion about what to include based on severity, based on discussions with @ashb, @kaxil and @jhtimmins.

I suspect there's maybe some more detail needed here between "here's what to pick" and "here's how to roll the RC", but maybe not?

This adds some more detail to the release README about the process of
selecting what is in a release, including the fact that the release
manager has some discretion about what to include based on severity.
Comment thread dev/README_RELEASE_AIRFLOW.md Outdated
@github-actions

Copy link
Copy Markdown
Contributor

The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide that full test matrix is needed and add the 'full tests needed' label. Then you should rebase it to the latest main or amend the last commit of the PR, and push it with --force-with-lease.

@github-actions github-actions Bot added the okay to merge It's ok to merge this PR as it does not require more tests label Aug 13, 2021
@andrewgodwin andrewgodwin marked this pull request as ready for review August 13, 2021 21:02
@andrewgodwin andrewgodwin requested a review from potiuk as a code owner August 13, 2021 21:02
@kaxil kaxil merged commit b7e5a24 into apache:main Aug 13, 2021
@kaxil kaxil deleted the release-readme branch August 13, 2021 21:18

@potiuk potiuk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Good description !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools okay to merge It's ok to merge this PR as it does not require more tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants