-
Notifications
You must be signed in to change notification settings - Fork 12
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
Automate releases on CI #78
Comments
I've been experimenting with this idea for |
Remember that this is an npm package, not an apm package. npm packages are straightforward to automate on travis: https://github.com/LiveSafe/gulp-html5-lint/blob/master/.travis.yml |
@SpainTrain my bad, saw the subject in my email and pounced. :D |
np, I've forgotten before also. |
Any particular benefits of doing this? |
You can't accidentally publish a release that isn't passing the builds, also you don't have to deal with it 😉. |
So here's the question, CI is triggered for PRs, if somebody bumps the version in a PR, would that publish a release? |
@steelbrain You can trigger it for new tags only. |
That's appealing |
Right? I've done it with RubyGems before, but then I set it to only release pre-releases and I would push full releases locally. |
Deployments and releases of any software should be automated because:
The best approach IMO is for CI to handle bumping, tagging, and publish/deploy. The way this generally works is:
I have used the approach of bumping in a PR and advise against using it. I have found the approach above to be easier to find/trace in commit logs, less error-prone, and faster/easier. |
Hello from 2 years into the future! semantic-release solves this very elegantly with npm packages now. |
Hah! That does solve it here since this is just an NPM package, not APM like the actual providers. |
No description provided.
The text was updated successfully, but these errors were encountered: