Skip to content
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 more detailed instructions on how to check pre-release candidates #32948

Merged

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Jul 30, 2023

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.
TESTING.rst Outdated Show resolved Hide resolved
TESTING.rst Outdated Show resolved Hide resolved
dev/README_RELEASE_AIRFLOW.md Outdated Show resolved Hide resolved
TESTING.rst Outdated Show resolved Hide resolved
TESTING.rst Outdated Show resolved Hide 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
Copy link
Contributor

@Adaverse Adaverse left a comment

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.

TESTING.rst Outdated Show resolved Hide resolved
TESTING.rst Outdated Show resolved Hide resolved
TESTING.rst Outdated Show resolved Hide resolved
TESTING.rst Outdated Show resolved Hide resolved
TESTING.rst Outdated Show resolved Hide resolved
Copy link
Member

@hussein-awala hussein-awala left a comment

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
42 checks passed
@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