Skip to content

Add more detailed instructions on how to check pre-release candidates - #32948

Merged
potiuk merged 5 commits into
apache:mainfrom
potiuk:add-description-on-how-to-test-release-candidates
Jul 30, 2023
Merged

potiuk merged 5 commits into
apache:mainfrom
potiuk:add-description-on-how-to-test-release-candidates

Conversation

@potiuk

@potiuk potiuk commented Jul 30, 2023

Copy link
Copy Markdown
Member

We ask our contributors to test pre-release candidates for Airflow and providers, but not everyone knows that Breeze allows to easily mix&match versions of airflow and providers that can be installed when running breeze shell or breeze start-airflow - which helps to test release candidates manually very, very quickly.

This change adds more detailed explanation and examples showing how it can be done, and links to those instruction from our release announcements and release documentation.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

We ask our contributors to test pre-release candidates for Airflow
and providers, but not everyone knows that Breeze allows to easily
mix&match versions of airflow and providers that can be installed
when running `breeze shell` or `breeze start-airflow` - which
helps to test release candidates manually very, very quickly.

This change adds more detailed explanation and examples showing
how it can be done, and links to those instruction from our
release announcements and release documentation.
Comment thread TESTING.rst Resolved
Comment thread TESTING.rst Resolved
Comment thread dev/README_RELEASE_AIRFLOW.md Resolved
Comment thread TESTING.rst Resolved
potiuk and others added 2 commits July 30, 2023 15:08
Co-authored-by: Pankaj Koti <pankajkoti699@gmail.com>
Co-authored-by: Pankaj Koti <pankajkoti699@gmail.com>
Comment thread TESTING.rst Resolved
@potiuk
potiuk force-pushed the add-description-on-how-to-test-release-candidates branch from d0e9da6 to 3963955 Compare July 30, 2023 13:47

@Adaverse Adaverse left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice one! A doc was needed for testing the release candidates. Just a few nits.

Comment thread TESTING.rst Resolved
Comment thread TESTING.rst Resolved
Comment thread TESTING.rst Resolved
Comment thread TESTING.rst Resolved
Comment thread TESTING.rst Resolved
Co-authored-by: Akash Sharma <35839624+Adaverse@users.noreply.github.com>

@hussein-awala hussein-awala 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.

Very useful!

@potiuk
potiuk merged commit 143d5e5 into apache:main Jul 30, 2023
@potiuk
potiuk deleted the add-description-on-how-to-test-release-candidates branch July 30, 2023 16:33
potiuk added a commit to potiuk/airflow that referenced this pull request Jul 30, 2023
After merging apache#32948 I realised the links were pointing to
a sub-section and the name was not encouraging.

This PR fixes it as a very small follow-up.
potiuk added a commit that referenced this pull request Jul 30, 2023
After merging #32948 I realised the links were pointing to
a sub-section and the name was not encouraging.

This PR fixes it as a very small follow-up.
pankajkoti added a commit to astronomer/airflow that referenced this pull request Aug 7, 2023
With the header change in PR apache#32948, the link to Verify by
contributors in the Status of providers testing issue in broken.
Correct the same by pointing it to the right header.
pierrejeambrun pushed a commit that referenced this pull request Aug 7, 2023
With the header change in PR #32948, the link to Verify by
contributors in the Status of providers testing issue in broken.
Correct the same by pointing it to the right header.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants