Giter Club home page Giter Club logo

Comments (6)

bcoe avatar bcoe commented on August 22, 2024 3

I'm a -1 on this one. Even though it seems like this is probably a good habit to be in, I think it goes against the goal of our specification being as simple and flexible as possible if we start prescribing too many rules about how commit messages should be written. tldr; it adds a step without adding much value IMO.

Perhaps we could compromise and add this to the F.A.Q. section?

should I use the imperative preset tense?

several communities recommend this and it might be a practice worth using.

In our redesign @damianopetrungaro maybe we can make our F.A.Q. a bit better to read and link to?


unrelated, @hbetts perhaps we should all work together and make sure we're not deviating from RFC 2119, I'd like to fix this.

from conventionalcommits.org.

hutson avatar hutson commented on August 22, 2024 1

flexible as possible

Making a specification flexible seems like an antithesis of the goal of a specification.

it adds a step without adding much value IMO.

This is definitely how I feel we should be gauging the value of adding an item to the specification.

I'm somewhat on the fence about whether the value outweighs the extra step/recommendation.

I find that the imperative form ensures a commit message description clearly states the change made, which ultimately makes it easier to reason about what my expectations should be for a commit.

However, it's difficult to enforce and easy to not follow.

Perhaps we could compromise and add this to the F.A.Q. section?

I would be fine with that. Though, instead of adding a FAQ question for should I use the imperative preset tense?, maybe generalize it a little, like what writing form should I use?.

unrelated, @hbetts perhaps we should all work together and make sure we're not deviating from RFC 2119, I'd like to fix this.

What did you have in mind, and should we file a separate issue to carry on that discussion?

from conventionalcommits.org.

bcoe avatar bcoe commented on August 22, 2024 1

What did you have in mind, and should we file a separate issue to carry on that discussion?

Yeah let's start with separate issues, I appreciate the set of eyes, I bet we've drifted a bit from the RFC over time; I was fairly new to this when I hammered out the first draft with @jameswomack.

making a specification flexible seems like an antithesis of the goal of a specification.

Fair point, the very act of making a specification is pretty pedantic 😝My major goals drafting the original version of this was to:

  1. have a central place to point to for the standard (the documentation was spread out over a few tools).
  2. distill the standard into as few steps as possible, such that it's easy to communicate to engineers that it isn't hard to add to your workflow (this is why I dropped some other requirements of angular, e.g., continuing to allow alternative types).

tldr; I'm trying to be resistant towards having too many steps if we can avoid it, which was why I leaned towards the F.A.Q. here.

from conventionalcommits.org.

damianopetrungaro avatar damianopetrungaro commented on August 22, 2024

I am totally fine with this!
I use this already as a best practice in all my commit messages, thanks to @apetro to point it out!

If someone wanna work on it for me it's fine.

@conventional-commits/committee what do you think about it?

from conventionalcommits.org.

damianopetrungaro avatar damianopetrungaro commented on August 22, 2024

@bcoe I agree with you and with @hbetts as well.

I think a good compromise would be using the FAQ and in the new UI I can do the section more relevant adding a link on the top of the page.

from conventionalcommits.org.

bcoe avatar bcoe commented on August 22, 2024

I think we covered this in the FAQ \o/

from conventionalcommits.org.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo 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.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.