-
Notifications
You must be signed in to change notification settings - Fork 32
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
feat: Enable GitHub actions. #150
Conversation
Changes: 1. Added workflow file 2. Added scripts to update the changelog in cli-core 3. Added the actions to bump the version and release to GitHub
package.json
Outdated
}, | ||
"release": { | ||
"branches": [ | ||
"github_action_changes" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to change the branch name to main
before merging.
.github/workflows/release.yml
Outdated
id: getTag | ||
run: | | ||
echo "::set-output name=TAG_NAME::$(git describe --tags $(git rev-list --tags --max-count=1))" | ||
- run: echo "${{steps.getTag.outputs.TAG_NAME}}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ignore the echo
statement, will remove it.
.github/workflows/release.yml
Outdated
change-log: ${{ steps.update-specs.outputs.change-log }} | ||
version-type: ${{ steps.update-specs.outputs.version-type }} | ||
steps: | ||
- name: Create temporary folder for copying json files fom OAI repo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: fom
.github/workflows/release.yml
Outdated
cache: 'npm' | ||
- name: semanticRelease | ||
run: | | ||
npm install --save-dev @semantic-release/changelog @semantic-release/git |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will remove this.
package.json
Outdated
@@ -59,5 +59,41 @@ | |||
}, | |||
"engines": { | |||
"node": ">=10.12.0" | |||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add @semantic-release/changelog @semantic-release/git
dev dependency.
da13195
to
bc2dccf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work! 🚀
Fixes
Closes: [DII-106, DII-107]
Dependent PR: CLI
Enable the GitHub actions.
Changes:
Testing Gist: Link
Checklist
If you have questions, please file a support ticket, or create a GitHub Issue in this repository.