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

WIP: fix: update to latest Angular commit conventions #220

Conversation

marionebl
Copy link

@marionebl marionebl commented Nov 20, 2017

@marionebl marionebl changed the title fix: update to latest commit conventions WIP: fix: update to latest commit conventions Nov 20, 2017
@coveralls
Copy link

coveralls commented Nov 20, 2017

Coverage Status

Coverage remained the same at 99.679% when pulling a2b882e on marionebl:fix/update-angular-conventions into 709dae1 on conventional-changelog:master.

@stevemao
Copy link
Member

stevemao commented Nov 20, 2017

@bcoe It's pretty hard to keep up-to-date with the angular convention without any help from the angular team. I was hoping to work with them on the specs. I think the specs should be centralised at https://conventionalcommits.org/ EG: they should probably hold discussions there, then change their docs. I tried to communicate with them multiple times but no luck. Any idea what would be the right channel to work with them?

@bcoe
Copy link
Member

bcoe commented Nov 21, 2017

@stevemao I agree; I see no reason to move away from chore it would break the workflow I've convinced many folks at npm, and in communities elsewhere to take on. Instead I'd rather that we have a new conventional commit type that we use by default for standard-version, which would always track https://conventionalcommits.org/?

What do you think of this @marionebl, we could still continue to keep the angular format up-to-date in the corresponding repo, but standard-verison would officially be adhering to a different set of guidelines.

@bcoe
Copy link
Member

bcoe commented Nov 21, 2017

... I could imagine landing this still to unblock folks ... but I would like to make an effort to diverge from the stricter angular guidelines.

@marionebl
Copy link
Author

marionebl commented Nov 21, 2017

Edit: There is no immediate action required for unblocking from my point of view - I moved the latest tag back to the compatible 4.3.0 version of @commitlint/config-angular.

I track everything related to this here: https://github.com/marionebl/commitlint/issues/146


@bcoe I'd be fine with ways forward that remove the current uncertainty about the exact conventions to use when talking "Angular" conventions.

I can see the following ToDos:

  1. Define the convention standard-version wants to track long-term.
  2. Create @commitlint/config-standard
  3. Create conventional-changelog/conventional-changelog-standard
  4. Adapt conventional-commits.org accordingly
  5. Reflect the breaking changes in all things *-angular via semver
  6. Instruct commitlint users to switch to @commitlint/config-standard by default
  7. Switch to conventional-changelog/conventional-changelog-standard by default
  8. Long tail: Do lots of PRs to rename to standard commit convention if that is what is in use

Doing a "fork" and renaming seems to be the safest way forward. As I see it that actually happened silently when the Angular team did their changes in January.

@marionebl marionebl changed the title WIP: fix: update to latest commit conventions WIP: fix: update to latest Angular commit conventions Nov 21, 2017
@bcoe
Copy link
Member

bcoe commented Nov 28, 2017

@marionebl this plan of action sounds wonderful (as always sorry for the slow reply, it's reached the point where I get dozens of issues opened on various open-source projects ever week; I'm trying to figure out a workflow that helps me keep up better).

Any interest in taking point on this project (getting us over to @commitlint/config-standard/conventional-changelog/conventional-changelog-standard)? I think this is a great future direction for this project.

A good place to reach me for coordinating work is this slack.

@bcoe bcoe added the triaged label Nov 28, 2017
@marionebl
Copy link
Author

@bcoe No worries/pressure it is your free time after all. :)

I'll drive the needed changes and ping you via Slack if needed:

  • publish @commitlint/config-conventional
  • agree on exact commit conventions (Slack)
  • adapt @commitlint/config-conventional as needed
  • rename/alias @commitlint/config-conventional to @commitlint/config-standard?
  • publish conventional-changelog/conventional-changelog-standard and use as default preset
  • adapt http://conventionalcommits.org/ to agreed commit conventions
  • major release for conventional-changelog-angular, reflecting the chore removal

@stevemao
Copy link
Member

@marionebl I made you an owner of this org so you can merge/close PRs/issues etc. Thanks!

@satazor
Copy link

satazor commented Nov 30, 2017

Awesome plan.

I prefer config-conventional but I’m fine with standard too.

@marionebl
Copy link
Author

@stevemao Thank you :)

@satazor
Copy link

satazor commented Dec 13, 2017

@marionebl I've tried using @commitlint/config-conventional but it still fails to work with standard-version:

⧗ input: chore(release): 1.0.0
✖ subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case]

I guess the issue is that the sentence starts with a number which is causing a validation error.
This works:

⧗ input: chore(release): v1.0.0
✔ found 0 problems, 0 warnings

How do you suggest to fix this? Should this be fixed in the config conventional?

@marionebl
Copy link
Author

@satazor Please raise this issue at commitlint, seams to be a regression with ensure-case in core.

@satazor
Copy link

satazor commented Dec 13, 2017

@bcoe
Copy link
Member

bcoe commented Dec 14, 2017

@marionebl awesome work creating config-conventional 👍

@bcoe
Copy link
Member

bcoe commented May 21, 2018

@marionebl what would it take for us to dust this off and land it?

@bcoe
Copy link
Member

bcoe commented Nov 1, 2018

cleaning up some old pull requests, please feel free to reopen this conversation if you're still interested in implementing this.

@marionebl I would still love to see us move to a conventional commits preset; so please feel free to dust this off at any time.

@bcoe bcoe closed this Nov 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

5 participants