Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Strenghten a bit and clarify importance of triaging issues #27262

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Oct 25, 2022

Triaging issues is an important part of the work we do. It is crucial that we respond swiftly and accurately to issues and discussions created by our users so that they feel welcome and listened to.

By participation in triaging process, contributors might show not only their willingness of helping other community members, but also show that they understand various parts of airflow (also that they are capable of learning it while helping others - which is one of the best ways to learn parts of Airflow you do not actively contribute to.


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

Copy link
Contributor

@rossturk rossturk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think these are good additions, Jarek 👍

Copy link
Contributor

@o-nikolas o-nikolas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great additions @potiuk!

COMMITTERS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
@potiuk potiuk force-pushed the strengthen-triage-process-importance-in-committership-path branch from 968cb3d to ac00396 Compare October 26, 2022 01:19
@potiuk potiuk marked this pull request as ready for review October 26, 2022 01:40
@potiuk
Copy link
Member Author

potiuk commented Oct 27, 2022

Any more comments?

Copy link

@shubham22 shubham22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great addition, thanks for formalizing this!

COMMITTERS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
@potiuk potiuk force-pushed the strengthen-triage-process-importance-in-committership-path branch 2 times, most recently from da95f6c to 64eda21 Compare October 29, 2022 12:05
Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looking good (just a few nitpicks).

I find this really helpful and I like the user centric approach.

ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
ISSUE_TRIAGE_PROCESS.rst Outdated Show resolved Hide resolved
Triaging issues is an important part of the work we do. It is crucial
that we respond swiftly and accurately to issues and discussions
created by our users so that they feel welcome and listened to.

By participation in triaging process, contributors might show not
only their willingness of helping other community members, but also
show that they understand various parts of airflow (also that they
are capable of learning it while helping others - which is one of
the best ways to learn parts of Airflow you do not actively
contribute to.
@potiuk potiuk force-pushed the strengthen-triage-process-importance-in-committership-path branch from 28053b9 to 6301d64 Compare October 29, 2022 15:19
@potiuk
Copy link
Member Author

potiuk commented Oct 29, 2022

Applied all comments from @pierrejeambrun @shubham22 as well and rebased. Let's see if others will chime in after pinging in the devlist.

@potiuk
Copy link
Member Author

potiuk commented Oct 29, 2022

I have a feeling that the title of the PR/Commit should be changed after all those comments :)

@potiuk
Copy link
Member Author

potiuk commented Nov 6, 2022

Merging as is. We can always improve it later.

@potiuk potiuk merged commit c185752 into apache:main Nov 6, 2022
@potiuk potiuk deleted the strengthen-triage-process-importance-in-committership-path branch November 6, 2022 16:16
@ephraimbuddy ephraimbuddy added the type:misc/internal Changelog: Misc changes that should appear in change log label Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:misc/internal Changelog: Misc changes that should appear in change log
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants