justinpakzad commented on code in PR #72746:
URL: https://github.com/apache/airflow/pull/72746#discussion_r4383149031
##########
scripts/ci/prek/check_airflowctl_command_coverage.py:
##########
@@ -101,7 +112,12 @@ def main():
missing = []
for group, subcommands in sorted(available.items()):
for subcommand in sorted(subcommands):
- cmd = f"{group} {subcommand}"
+ subgroup = next((sg for sg in SUBGROUPS.get(group, ()) if
subcommand.endswith(sg)), None)
Review Comment:
Updated to use `endswith(f"-{sg}")`. I didn't see a way around the duplicate
logic since we can't import a shared helper from `airflow-ctl` into the prek
directory.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
