Skip to content

Refactor package checks to be test and remove duplication - #44318

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:slightly-refactor-provider-tests
Nov 24, 2024
Merged

Refactor package checks to be test and remove duplication#44318
potiuk merged 1 commit into
apache:mainfrom
potiuk:slightly-refactor-provider-tests

Conversation

@potiuk

Copy link
Copy Markdown
Member

Package checks are already "tests" not only checks since we introduced compatibility complete tests. This PR updates names of files and also removes duplication of wheel/sdist package build and verification and adding conditional steps.


^ 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.

@potiukpotiuk added the full tests needed We need to run full set of tests for this PR to merge label Nov 24, 2024
@potiuk
potiukforce-pushed the slightly-refactor-provider-tests branch from bd3fbd2 to 257371dCompareNovember 24, 2024 01:16
Package checks are already "tests" not only checks since we
introduced compatibility complete tests. This PR updates
names of files and also removes duplication of wheel/sdist
package build and verification and adding conditional steps.
@potiuk
potiukforce-pushed the slightly-refactor-provider-tests branch from 257371d to 4cb90eeCompareNovember 24, 2024 09:55
@potiuk
potiuk merged commit 18d1b97 into apache:mainNov 24, 2024
@potiuk
potiuk deleted the slightly-refactor-provider-tests branch November 24, 2024 11:34
got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
Package checks are already "tests" not only checks since we
introduced compatibility complete tests. This PR updates
names of files and also removes duplication of wheel/sdist
package build and verification and adding conditional steps.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-toolsfull tests neededWe need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@potiuk@amoghrajesh@eladkal