airflow icon indicating copy to clipboard operation
airflow copied to clipboard

Move prohibited warnings detection in deferrable pytest plugin

Open Taragolis opened this issue 1 year ago • 1 comments

related: https://github.com/apache/airflow/issues/38642

Move all logic around existed functionality into the separate plugin, which allow to do additional things without turned root conftest.py into the mess

  • Allow to disable detect warnings locally
  • Move list of prohibited warnings into the pyproject.toml
  • Analyse test failure and if it due to listed prohibited warning show hint to the developer

image


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

Taragolis avatar May 05 '24 00:05 Taragolis

I'm taking myself off the reviewer list as I have no clue about what this plugin is about and hope others do :-D

jscheffl avatar May 05 '24 07:05 jscheffl