Skip to content

chore: Add changelog using standard-changelog #696

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

Closed
wants to merge 3 commits into from

Conversation

machour
Copy link
Member

@machour machour commented Jan 16, 2018

Description

Reopening #504 using an automated tool, let's do this overdue changelog thing! 🎉

@coveralls
Copy link

Coverage Status

Coverage remained the same at 43.31% when pulling 88deb17 on machour:add-changelog into 608818e on gitpoint:master.

@machour
Copy link
Member Author

machour commented Jan 16, 2018

Here's how the ChangeLog looks for now: https://gist.github.com/machour/6a6e41ec696a198021fb613f10e58a02

I generated it using ./node_modules/.bin/standard-changelog --first-release then edited the result according to my archeologic researches made in #504.

Still need to :

  • Polish the generated file
  • Add the links for < 1.3.0
  • Wait for a few PRs to get merged and sync the file to see how it goes
  • Document the proper workflow for releases (I think we need to git tag && git push --tags before re-generating the file)

@housseindjirdeh
Copy link
Member

Amazing - how does the library work? Do we need to re-run the command everytime a commit is merged?

@machour
Copy link
Member Author

machour commented Jan 16, 2018

I'm still unclear about the workflow, but I think this would have to be ran once per release.

@machour
Copy link
Member Author

machour commented Jan 24, 2023

Pfiou, time flies :)
Closing this irrelevant PR.

@machour machour closed this Jan 24, 2023
@machour machour deleted the add-changelog branch January 24, 2023 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants