-
Notifications
You must be signed in to change notification settings - Fork 617
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: add automation for docker image #63
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,35 @@ | ||||||||||
name: Semantic Release | ||||||||||
on: | ||||||||||
push: | ||||||||||
branches: | ||||||||||
- main | ||||||||||
|
||||||||||
jobs: | ||||||||||
Release: | ||||||||||
runs-on: ubuntu-latest | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This token permission is set by default but could be useful to state it explicitly.
Suggested change
|
||||||||||
steps: | ||||||||||
- name: Checkout code | ||||||||||
uses: actions/checkout@v3 | ||||||||||
|
||||||||||
- name: Setup Node.js | ||||||||||
uses: actions/setup-node@v3 | ||||||||||
with: | ||||||||||
node-version: '18' | ||||||||||
|
||||||||||
- name: Cache Node.js modules | ||||||||||
uses: actions/cache@v3 | ||||||||||
with: | ||||||||||
path: ~/.npm | ||||||||||
key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }} | ||||||||||
restore-keys: | | ||||||||||
${{ runner.os }}-node- | ||||||||||
|
||||||||||
- name: Install dependencies | ||||||||||
run: npm ci | ||||||||||
|
||||||||||
- name: Semantic Release | ||||||||||
env: | ||||||||||
GITHUB_TOKEN: ${{ secrets.GA_TOKEN }} | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You should use the default token created for the workflow and adjust the permissions assigned to it instead.
Suggested change
|
||||||||||
DOCKER_REGISTRY_USER: ${{ secrets.DOCKER_REGISTRY_USER }} | ||||||||||
DOCKER_REGISTRY_PASSWORD: ${{ secrets.DOCKER_REGISTRY_PASSWORD }} | ||||||||||
Comment on lines
+33
to
+34
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think that these environment variables are named incorrectly,
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. They'll also need to be configured in this repository prior to merge! |
||||||||||
run: npx semantic-release |
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.
Suggest using the
name
key to explicitly assign a name to the job instead.