-
Notifications
You must be signed in to change notification settings - Fork 145
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
remove husky and use CI #661
Changes from 2 commits
93e4e3e
daf3474
64b08bb
6707bbb
8d2b8ce
34ab028
d78199d
dd051da
a696d93
3efd07b
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 |
---|---|---|
|
@@ -6,9 +6,82 @@ env: | |
BASE_IMAGE: ghcr.io/graphprotocol/graph-docs-staging | ||
|
||
jobs: | ||
lint: | ||
if: contains(github.head_ref, 'crowdin') == false | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Cancel Previous Runs | ||
uses: styfle/cancel-workflow-action@0.9.1 | ||
with: | ||
access_token: ${{ github.token }} | ||
|
||
- name: Check out the repo | ||
uses: actions/checkout@v4 | ||
|
||
- name: Set up env | ||
uses: the-guild-org/shared-config/setup@main | ||
with: | ||
nodeVersion: 20 | ||
packageManager: pnpm | ||
|
||
- name: Install dependencies | ||
run: pnpm install | ||
|
||
- name: Lint | ||
run: pnpm lint | ||
|
||
format: | ||
if: contains(github.head_ref, 'crowdin') == false | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Cancel Previous Runs | ||
uses: styfle/cancel-workflow-action@0.9.1 | ||
with: | ||
access_token: ${{ github.token }} | ||
|
||
- name: Check out the repo | ||
uses: actions/checkout@v4 | ||
|
||
- name: Set up env | ||
uses: the-guild-org/shared-config/setup@main | ||
with: | ||
nodeVersion: 20 | ||
packageManager: pnpm | ||
|
||
- name: Install dependencies | ||
run: pnpm install | ||
Comment on lines
+51
to
+52
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. Are we gonna spend more Actions minutes since this now runs 3 times instead of once? |
||
|
||
- name: Format | ||
saihaj marked this conversation as resolved.
Show resolved
Hide resolved
|
||
run: pnpm prettier:check | ||
|
||
typecheck: | ||
if: contains(github.head_ref, 'crowdin') == false | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Cancel Previous Runs | ||
uses: styfle/cancel-workflow-action@0.9.1 | ||
with: | ||
access_token: ${{ github.token }} | ||
|
||
- name: Check out the repo | ||
uses: actions/checkout@v4 | ||
|
||
- name: Set up env | ||
uses: the-guild-org/shared-config/setup@main | ||
with: | ||
nodeVersion: 20 | ||
packageManager: pnpm | ||
|
||
- name: Install dependencies | ||
run: pnpm install | ||
|
||
- name: Typecheck | ||
run: pnpm typecheck | ||
|
||
build: | ||
if: contains(github.head_ref, 'crowdin') == false | ||
runs-on: ubuntu-latest | ||
needs: [lint, format, typecheck] | ||
saihaj marked this conversation as resolved.
Show resolved
Hide resolved
|
||
steps: | ||
- name: Cancel Previous Runs | ||
uses: styfle/cancel-workflow-action@0.9.1 | ||
|
This file was deleted.
This file was deleted.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
This is currently failing on this PR, not sure why. 🤔
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.
@saihaj – No idea why I had to make these ugly changes to make
pnpm lint
pass in CI (it works without those changes locally).