-
Notifications
You must be signed in to change notification settings - Fork 31
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
make changelog requirement less strict #35
Comments
switch to warnings, e.g.
|
Is this still active? |
Yes! This would be a major (breaking) change, and requires a little consideration before implementing, but would be a great addition to current functionality. My concern is avoiding breaking workflows for current users who rely on the current CLI's strictness to ensure deploys are successful. Maybe hiding this behind a flag (like |
@ungoldman Hey hey. Sorry about the delay in responding to this. Happy to help out if possible, although I think I need a deeper understanding of all the moving parts before I can offer anything of value. I'll dig into the internals and get back to you ASAP. That said, your solution sounds like a good one! |
Is this still a requested feature? Was thinking of implementing it bc we have a case where no changelog is necessary. Also was thinking of adding a way to auto advance the package version number and commit it so the entire process of bumping the release and releasing it can be automated. |
I would still like to see it. Also, it would be nice if there was a skip release option, for when you use a gh-release hook in the release nom script. Sometimes you already gh-released and just need to skip interactively. |
I ended up using release-it since it did everything I needed out of the box. If I free up again and have a need for a smaller release lib I'll look at it again thanks. |
maybe change it from an error to a warning and allow either an empty body or use https://www.npmjs.com/package/string-editor
The text was updated successfully, but these errors were encountered: