-
Notifications
You must be signed in to change notification settings - Fork 0
Sk/ssf 132 verify prettier check #99
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
base: main
Are you sure you want to change the base?
Changes from all commits
60beb3c
3e6fdeb
9e62a9a
0971ca1
80ce71f
92686b6
4d3d693
ec2fb63
c5c7b9b
294e85e
a474507
3cc1010
dc6aa8c
2f575a3
85ff2df
e7f9c02
5bd6ce2
4f0642c
bfcb073
d09f888
050c9aa
365a6d3
509a5dd
64fcf76
83edbbd
989fca2
39cf0e6
ec495cb
22aaa1b
e9d840f
535a691
731d759
ad23be3
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 |
|---|---|---|
| @@ -1,6 +1,16 @@ | ||
| on: [push, pull_request] | ||
|
|
||
| jobs: | ||
| prettier-check: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 20 | ||
| - run: yarn install | ||
| - run: yarn prettier:check | ||
|
|
||
| backend-tests: | ||
| runs-on: ubuntu-latest | ||
| services: | ||
|
|
@@ -30,7 +40,6 @@ jobs: | |
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 20 | ||
|
|
||
| - run: yarn install --frozen-lockfile | ||
| - run: yarn list strip-ansi string-width string-length | ||
| - run: yarn test | ||
| - run: npx jest | ||
|
Comment on lines
43
to
45
Member
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 these things should be moved to jobs from services |
||
|
Collaborator
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 adding a small section on CI would be nice: CI (GitHub Actions)
Local Commands
|
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.
since backend-tests use
yarn install --frozen-lockfilelets also use that here to make sure ci & local env versions stays consistent