add release config #38

Open
philippdormann wants to merge 3 commits from philippdormann/feature/release-management into master
philippdormann commented 4 years ago (Migrated from github.com)
Owner
  • automated changelog containing commits + diff from last release
  • automated version bumping
  • tag creation
    • might be used to trigger GitHub workflows/ GitHub Actions for Docker releases in the future?
  • semver support
- automated changelog containing commits + diff from last release - automated version bumping - tag creation - might be used to trigger GitHub workflows/ GitHub Actions for Docker releases in the future? - semver support
Saibamen commented 4 years ago (Migrated from github.com)
Owner

@philippdormann: Please resolve conflicts

@louislam: +1 for this. This will take care of correct releases in GitHub.
Current release is wrong:
image

@philippdormann: Please resolve conflicts @louislam: +1 for this. This will take care of correct releases in GitHub. Current release is wrong: ![image](https://user-images.githubusercontent.com/905878/128191422-448b1ed2-67b4-441d-a218-378f976668ff.png)
philippdormann commented 4 years ago (Migrated from github.com)
Poster
Owner

thanks for stepping in and "+1'ing" 😉

sure thing :) will resolve the conflicts shortly

thanks for stepping in and "+1'ing" 😉 sure thing :) will resolve the conflicts shortly
Saibamen (Migrated from github.com) requested changes 4 years ago
Saibamen (Migrated from github.com) commented 4 years ago
Owner

Please move it to devDependencies. We don't want to have unused packaged in PROD Docker image (https://github.com/louislam/uptime-kuma/pull/122)

Please also add ^ before versions

Please move it to devDependencies. We don't want to have unused packaged in PROD Docker image (https://github.com/louislam/uptime-kuma/pull/122) Please also add `^` before versions
nicolaiort (Migrated from github.com) reviewed 4 years ago
nicolaiort (Migrated from github.com) commented 4 years ago
Owner

The two dependencies you mentioned are already devDependencies :)

The two dependencies you mentioned are already devDependencies :)
Saibamen (Migrated from github.com) approved these changes 4 years ago
Saibamen (Migrated from github.com) left a comment

LGTM

Saibamen (Migrated from github.com) reviewed 4 years ago
Saibamen (Migrated from github.com) commented 4 years ago
Owner

Sorry, I didn't see full file before :P

Sorry, I didn't see full file before :P
nicolaiort (Migrated from github.com) approved these changes 4 years ago
nicolaiort (Migrated from github.com) left a comment

the new release configs are nice

louislam commented 4 years ago (Migrated from github.com)
Owner

Since my release approach is a bit special, I have my own script for that. Will be back to this later.

Since my release approach is a bit special, I have my own script for that. Will be back to this later.
Saibamen commented 4 years ago (Migrated from github.com)
Owner
@philippdormann: Can you check https://github.com/louislam/uptime-kuma/blob/master/extra/update-version.js and https://github.com/louislam/uptime-kuma/blob/master/extra/mark-as-nightly.js to see how current release looks like?
philippdormann commented 4 years ago (Migrated from github.com)
Poster
Owner

well of course I took a look at it and
tbh don't like it at all - that's why this PR was created 😄😬

well of course I took a look at it and tbh don't like it at all - that's why this PR was created 😄😬
This pull request has changes conflicting with the target branch.
package-lock.json
package.json
Sign in to join this conversation.
Loading…
There is no content yet.