Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Doc: added triage section to new contributor docs#28347

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
rcomer merged 1 commit intomatplotlib:mainfromstory645:contribute-triage
Aug 19, 2024

Conversation

story645
Copy link
Member

@story645story645 commentedJun 5, 2024
edited
Loading

PR summary

Discussion from#28335 had me thinking that my issues w/ the guidance being in the wrong place was that triage is its own category of contribution and therefore it should get a section incontribute.html

This also incorporates the guidance from#28335 (particularly language/reasoning from#28335 (comment)) and therefore supercedes it.

PR checklist

@github-actionsgithub-actionsbot added the Documentation: devdocsfiles in doc/devel labelJun 5, 2024
@story645story645 marked this pull request as draftJune 5, 2024 02:18
@story645story645force-pushed thecontribute-triage branch 4 times, most recently from22ffb99 to4c8ba46CompareJune 5, 2024 20:29
@story645story645 marked this pull request as ready for reviewAugust 7, 2024 22:12
@story645
Copy link
MemberAuthor

I had made this draft out of deference to#28335, but also want a decision on this before work starts on#28476

Comment on lines 181 to 186
Triage help is a valuable contribution because it takes a lot of time and
judgement to effectively curate and resolve issues; therefore we very much
appreciate thoughtful feedback to issue authors. Please do not verbatim post
the output of a generative AI tool (such as chat GPT) as a response to a bug
report as we expect that the issue author has either already tried that or has a
reason for not doing so.
Copy link
Member

Choose a reason for hiding this comment

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

This is a way of working aspect of triaging. It belongs under the above linkedbug triaging docs, not here. - My idea of "Contributing guide > Getting started" is that it should give a high-level idea of the kinds contribution and direct interested people to the relevant parts of the documentation. Specific information on how (not) to do things should be in those parts of the documentation e.g.bug triaging, not on the overview page.

story645 reacted with thumbs up emoji
Copy link
MemberAuthor

Choose a reason for hiding this comment

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

I'll move it but I put it here b/c it's similar to the "no claiming" guidance that we have under "find an issue"https://matplotlib.org/devdocs/devel/contribute.html#work-on-an-issue

Which should I move that too?

Copy link
Member

@timhoffmtimhoffmAug 8, 2024
edited
Loading

Choose a reason for hiding this comment

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

My first impulse was: This belongs inhttps://matplotlib.org/devdocs/devel/triage.html. But considering the definition of triage actions on that site, replying with a solution is beyond triaging. Triaging only makes sure the issue is well laid out and sorted. It does not solve it (except for invalid or duplicate issues). Therefore, I don't see that people will respond with AI as part of triaging. To make life simple here, I claim it's out of scope for the PR and we can leave that section out.

In as separate PR, you could add something like "Proposing solutions" to thehttps://matplotlib.org/devdocs/devel/triage.html. It should be made clear that

  • it's optional (a triager may add it if they already exactly know how to solve this)
  • proposals / recommendations should be well considered and well laid out. If you are not sure, better leave the response to somebody else. This also includes auto-generated AI responses are not acceptable.

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

To make life simple here, I claim it's out of scope for the PR and we can leave that section out.

😭 that section was the whole motivation for this PR, but at least spinning it out into a new PR means this one no longer has to wait on the consensus on#28335

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

Also, separately I think "propose a solution" is requiredwhen somebody wants to close an issue claiming it's solved.

Copy link
Member

@timhoffmtimhoffm left a comment

Choose a reason for hiding this comment

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

I approve the content. Take or leave my comments. You may self-merge.

* debugging and resolving issues
* linking to related issues, discussion, and external work

Helping out with triage is discussed in detail in :ref:`bug_triaging`.
Copy link
Member

Choose a reason for hiding this comment

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

With my non-native English, I feel "helping out" is not that welcoming. I read it as "We have a deficiency. Can you temporarily take over a task that we currently cannot fulfill".

I suggest either

Suggested change
Helpingoutwith triage is discussed in detail in:ref:`bug_triaging`.
Helping with triage is discussed in detail in:ref:`bug_triaging`.

or

Suggested change
Helping out with triage is discussed in detail in:ref:`bug_triaging`.
These tasks are discussed in detail in:ref:`bug_triaging`.

But you know the language better, so please decide whether to keep as is or change.

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

But you know the language better, so please decide whether to keep as is or change.

I'll change it because your helpful review told me how what I wrote is being read & that it's not conveying what I wanted it to.

Copy link
MemberAuthor

@story645story645Aug 8, 2024
edited
Loading

Choose a reason for hiding this comment

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

Also changing it to

Our triage process is discussed in detail in :ref:bug_triaging

which I think ties in really well w/@rcomer's suggestion of

If you have any questions about the process, please reach out on the :ref:`contributor_incubator

Comment on lines 118 to 119
:class-card: sd-outline-{primary} sd-shadow-none
:class-body: sd-text-{primary}
Copy link
Member

Choose a reason for hiding this comment

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

These stylings seem to make the cards less responsive to mouse-over. There's only a minimal size/layout change:
https://output.circle-artifacts.com/output/job/5b25daa3-7646-458d-9b4d-a2b5ab7702fb/artifacts/0/doc/build/html/devel/index.html

Previously, at least the small frame changed to purple, indicating a link.
https://matplotlib.org/devdocs/devel/index.html

story645 reacted with thumbs up emoji
Copy link
MemberAuthor

Choose a reason for hiding this comment

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

Will change back but that seems like a pydata-sphinx-issue maybe :/

Co-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>Co-authored-by: Ruth Comer <10599679+rcomer@users.noreply.github.com>
@story645
Copy link
MemberAuthor

Made all the changes but tweaked enough that this should maybe get a 2nd look,@rcomer or@timhoffm

Copy link
Member

@rcomerrcomer left a comment

Choose a reason for hiding this comment

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

Sorry I only just noticed you pinged for another review here. Seems good to me 👍

story645 reacted with heart emoji
@rcomerrcomer added this to thev3.10.0 milestoneAug 19, 2024
@rcomerrcomer merged commit1c892c2 intomatplotlib:mainAug 19, 2024
22 checks passed
@story645story645 deleted the contribute-triage branchNovember 21, 2024 01:02
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@timhoffmtimhoffmtimhoffm approved these changes

@rcomerrcomerrcomer approved these changes

Assignees
No one assigned
Labels
Documentation: devdocsfiles in doc/devel
Projects
None yet
Milestone
v3.10.0
Development

Successfully merging this pull request may close these issues.

3 participants
@story645@timhoffm@rcomer

[8]ページ先頭

©2009-2025 Movatter.jp