consolidating the bug reports to one issue template#64240
consolidating the bug reports to one issue template#64240potiuk merged 24 commits intoapache:mainfrom
Conversation
a5ba367 to
d6d1548
Compare
d6d1548 to
c91da58
Compare
a309be3 to
21bf1a2
Compare
21bf1a2 to
1c60750
Compare
bugraoz93
left a comment
There was a problem hiding this comment.
Look good to me! It follows the proposed approach in the docs. Thanks for the PR!
shahar1
left a comment
There was a problem hiding this comment.
Looks good! Got a few comments, please take a look on you have the time.
Thanks!
- resolving few nits. - added Task SDK as a separate option under issue type.
|
@Shrividya This PR has a few issues that need to be addressed before it can be reviewed — please see our Pull Request quality criteria. Issues found:
What to do next:
There is no rush — take your time and work at your own pace. We appreciate your contribution and are happy to wait for updates. If you have questions, feel free to ask on the Airflow Slack. Note: This comment was drafted by an AI-assisted triage tool and may contain mistakes. Once you have addressed the points above, an Apache Airflow maintainer — a real person — will take the next look at your PR. We use this two-stage triage process so that our maintainers' limited time is spent where it matters most: the conversation with you. |
|
@shahar1 all the comments have been addressed, would you mind reviewing again so that the comments can be marked as resolved or if it needs further changes? |
shahar1
left a comment
There was a problem hiding this comment.
Overall LGTM, I'd rather have at least one more review by committer for merging.
Great job!
There was a problem hiding this comment.
Pull request overview
This PR consolidates multiple GitHub issue templates into a single Airflow issue report form, updates CI/pre-commit checks to point at the new template, and removes now-redundant templates.
Changes:
- Consolidated core/providers/Helm/Task SDK bug reporting into
.github/ISSUE_TEMPLATE/1-airflow_bug_report.ymlwith a new category dropdown. - Removed separate Providers / Helm chart / Docs issue templates and introduced a maintainer-only “Task and Meta” free-form template.
- Updated the pre-commit hook and its checker script to validate the provider list in the consolidated template.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
scripts/ci/prek/check_airflow_bug_report_template.py |
Points validation at the new consolidated issue template |
.pre-commit-config.yaml |
Updates the pre-commit hook to run against the new template |
.github/ISSUE_TEMPLATE/1-airflow_bug_report.yml |
Consolidates issue reporting, adds category/provider/Helm fields |
.github/ISSUE_TEMPLATE/3-free_form.yml |
Adds maintainer-only meta/task issue form |
.github/ISSUE_TEMPLATE/3-airflow_providers_bug_report.yml |
Removed (replaced by consolidated template) |
.github/ISSUE_TEMPLATE/4-airflow_helmchart_bug_report.yml |
Removed (replaced by consolidated template) |
.github/ISSUE_TEMPLATE/5-airflow_doc_issue_report.yml |
Removed |
|
thanks for the PR :) |
Backport failed to create: v3-1-test. View the failure log Run detailsNote: As of Merging PRs targeted for Airflow 3.X In matter of doubt please ask in #release-management Slack channel.
You can attempt to backport this manually by running: cherry_picker 3b6ee39 v3-1-testThis should apply the commit to the v3-1-test branch and leave the commit in conflict state marking After you have resolved the conflicts, you can continue the backport process by running: cherry_picker --continueIf you don't have cherry-picker installed, see the installation guide. |
Motivation for this PR is the conversation in the following links:
Changes include:
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.