Comments (10)
Clarifying question: does someone reading the commit need to know that a commit is a [suggested term]?
I think for typo
, the answer is yes - when I'm looking over a change history, it can be really useful to distinguish "this change literally only fixed an internal typo" from more significant/big changes of refactor
/docs
/style
/etc type. Usually when I go into commit history, I have a question for which typo fixes are irrelevant (and if mere typo fixes are relevant, I know from the nature of the question I have, before I start looking).
I think for nit
, the answer is no - if the change mattered enough to go in, tell future readers why/how. What are you nitting? Does the nit cover some edge case, no matter how tiny/insignificant/unlikely? fix
! Does the nit enhance the behavior in a nice-to-have way? feat
! Neither of those but changes code? refactor
(or style
, if the justification is purely that it makes the code more pleasing). Is it a wording nit in a doc/README/comment? docs
! And so on.
I think for "proofreading", I can't answer, because I don't know what "proofreading" means here. You're clearly using it in a more specialized way, and maybe you're onto a really useful distinction of change type, but I don't know what it is. (If you can define/explain it, I can probably suggest a better word, especially if I see the usefulness.)
from conventionalcommits.org.
Cheers, thanks for clarifying!
I do make a similar distinction in my commits - for years I've been using "wording {fix,tweak}" for human language / prose adjustments.
Thinking about Conventional Commits and the related convention from Angular, maybe the distinction that matters here is just "is this a purely stylistic change?" without the additional bit of "is this a code change or prose change?" (if the grammar error isn't impactful enough to earn a
fix
ordocs
type,style
is probably fine).But if I was version-controlling a specification or book, for example, I would absolutely want to distinguish prose changes from more impactful fixes. A perfect use-case for having a communal CCRFI process/repo ( #537 )! Personally, I propose
wording
orlanguage
as a general catch-all (idk if book editing use-cases would want to get more granular).
Now that I think about it @mentalisttraceur, wording
or language
does seem more fitting.
from conventionalcommits.org.
Oh, to be clear I don't have any special authority here. I'm just a guy who noticed Conventional Commits recently, here to contribute my ideas and thinking to help this convention and community succeed at its full potential.
I am glad I was able to help you with your proposal, and I'm really hoping that my advocacy for #537 helps create a community norm and place which allows proposals like yours to flourish as extensions, gaining visibility and adoption in as much as they're good/helpful! That's all I can do. :)
from conventionalcommits.org.
Oh, to be clear I don't have any special authority here. I'm just a guy who noticed Conventional Commits recently, here to contribute my ideas and thinking to help this convention and community succeed at its full potential.
I am glad I was able to help you with your proposal, and I'm really hoping that my advocacy for #537 helps create a community norm and place which allows proposals like yours to flourish as extensions, gaining visibility and adoption in as much as they're good/helpful! That's all I can do. :)
Oh thanks for clarifying @mentalisttraceur and what a great proposal you have there. :)
from conventionalcommits.org.
@mentalisttraceur in #529 (comment)
I think for
typo
, the answer is yes - when I'm looking over a change history, it can be really useful to distinguish "this change literally only fixed an internal typo" from more significant/big changes of refactor/docs/style/etc type.
I buy this rationale and I also miss the typo
type of commits.
from conventionalcommits.org.
But eventually these will land you to better open-source contributor.
Proofreading and typos are fine, but nitpicking is not something you need to look for.
from conventionalcommits.org.
P.S. Note that Conventional Commits v1.0.0 only defines fix
and feat
(feature) change types, but it allows other types - so this seems like the kind of thing best served by defining your own additional convention which extends Conventional Commits by defining one or more change types.
The Conventional Commits community would probably benefit a lot from mimicking something like Scheme's "Request for Implementation" process for stuff like this.
from conventionalcommits.org.
Clarifying question: does someone reading the commit need to know that a commit is a [suggested term]?
I think for
typo
, the answer is yes - when I'm looking over a change history, it can be really useful to distinguish "this change literally only fixed an internal typo" from more significant/big changes ofrefactor
/docs
/style
/etc type. Usually when I go into commit history, I have a question for which typo fixes are irrelevant (and if mere typo fixes are relevant, I know from the nature of the question I have, before I start looking).I think for
nit
, the answer is no - if the change mattered enough to go in, tell future readers why/how. What are you nitting? Does the nit cover some edge case, no matter how tiny/insignificant/unlikely?fix
! Does the nit enhance the behavior in a nice-to-have way?feat
! Neither of those but changes code?refactor
(orstyle
, if the justification is purely that it makes the code more pleasing). Is it a wording nit in a doc/README/comment?docs
! And so on.I think for "proofreading", I can't answer, because I don't know what "proofreading" means here. You're clearly using it in a more specialized way, and maybe you're onto a really useful distinction of change type, but I don't know what it is. (If you can define/explain it, I can probably suggest a better word, especially if I see the usefulness.)
Thanks for your feedback @mentalisttraceur. To elaborate, proofreading is something I came up with when reviewing pull requests with recurring grammar errors(note: I come from a background in teaching so that also plays an influence in how I formulated this review tactic.
from conventionalcommits.org.
Cheers, thanks for clarifying!
I do make a similar distinction in my commits - for years I've been using "wording {fix,tweak}" for human language / prose adjustments.
Thinking about Conventional Commits and the related convention from Angular, maybe the distinction that matters here is just "is this a purely stylistic change?" without the additional bit of "is this a code change or prose change?" (if the grammar error isn't impactful enough to earn a fix
or docs
type, style
is probably fine).
But if I was version-controlling a specification or book, for example, I would absolutely want to distinguish prose fixes from other kinds of fixes. A perfect use-case for having a communal CCRFI process/repo ( #537 )! Personally, I propose wording
or language
as a general catch-all (idk if book editing use-cases would want to get more granular).
from conventionalcommits.org.
Now that's settled! :) Do I have your permission to go forth and make my contribution @mentalisttraceur?
from conventionalcommits.org.
Related Issues (20)
- e62041900a74d076e1462f7a3e900eb5d7a513e2 HOT 2
- Sign up
- How to configre conventional-changelog-conventionalcommits 7? HOT 1
- `feat` is contracted, most other commonly used tags are full words. HOT 9
- 6_make test.txt
- fix: navigation bar is not responsive on smaller devices HOT 1
- Delete the .idea folder HOT 1
- Decoupling from specific versioning formats (i.e. SemVer) HOT 1
- what is the recommended way to handle deprecations? HOT 2
- Request for Arabic Language Support
- How do I choose a right commit type for adding media content?
- Specification doesn't permit footers without a body HOT 2
- Support Hindi Language HOT 1
- Add clarification for description formatting
- Scheme for documentation
- Should there be (a11y) commit type? How do you address accessibility UI related changes? HOT 1
- GPG key not found HOT 2
- Bad certificate on www.conventionalcommits.org HOT 10
- Ambiguity over footer issue references
- Do we need special types for GitOps commits ?
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from conventionalcommits.org.