Zulip Chat Archive
Stream: mathlib4
Topic: bikeshed: Labeling linter PRs with "feat"?
Michael Rothgang (May 27 2024 at 08:07):
In #13220, a question came up between @Yaël Dillies and me: how to label a PR adding a new linter? There are two reasonable positions here:
- "feat" is only for new mathematical content, so this should be labelled something like "chore"
- This is a new feature in the "linters" category. (We even have a "t-lint" label now.) So, this should be labelled "feat".
Any opinions on this bikeshed?
Mario Carneiro (May 27 2024 at 08:09):
This is obviously a "feat"
Mario Carneiro (May 27 2024 at 08:09):
there has never been a rule that "feat" is only for maths
Yaël Dillies (May 27 2024 at 08:10):
My reaction was mostly because I've seen clearly chore
PRs being labelled as feat
recently, so I was a bit worried people understood the convention to be "Start your PR title with feat:
"
Mario Carneiro (May 27 2024 at 08:12):
not sure how to comment on that without knowing what the "clearly chore
PRs" are. What is "clearly chore
" to you?
Yaël Dillies (May 27 2024 at 08:19):
eg #13015, #13002, #12431, #12423, #12289, #12023, #11980, ...
Yaël Dillies (May 27 2024 at 08:19):
Not claiming all should be chore
. Most of them should be refactor
instead
Michael Rothgang (May 27 2024 at 08:44):
Arguably, generalising a proof could be a feature - but for mathematically trivial generalisations, using refactor
seems better indeed.
Eric Wieser (May 27 2024 at 10:54):
I was under the impression that refactor
was for non-trivial generalizations
Last updated: May 02 2025 at 03:31 UTC