The GitHub Actions job "Tests (AMD)" on 
airflow.git/fix-airflowctl-config-lint-substring-match has succeeded.
Run started by GitHub user rjgoyln (triggered by rjgoyln).

Head commit for run:
0fa3e39267a542b219f762152754429ebf22ed70 / rjgoyln <[email protected]>
Fix airflowctl config lint filters matching partial config names

The lint filters were declared as plain strings, so the membership tests in
the command body compared a name against a string rather than against a list.
Any name that happened to be a prefix of what the user typed slipped through:
`--option auth_backends` also reported `auth_backend`, and the mirror case on
`--ignore-option` silently suppressed warnings the user still wanted to see.
Five such pairs exist in CONFIGS_CHANGES today.

The comma-separated lists the docstring has always advertised only appeared to
work because substring matching happened to accept them; the verbose output
joined the same strings character by character.

airflow-core already parses these four arguments with string_list_type, which
was defined here but left unused.

Report URL: https://github.com/apache/airflow/actions/runs/30748478232

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to