ci: migrate to GitHub Actions #286
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| version: 2 | ||
| updates: | ||
| # Set update schedule for GitHub Actions | ||
| - package-ecosystem: 'github-actions' | ||
| directory: '/' | ||
| schedule: | ||
| interval: 'daily' |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,69 @@ | ||
| name: CI | ||
|
|
||
| on: | ||
| push: | ||
| branches: | ||
| # semantic-release valid branches | ||
| - '+([0-9])?(.{+([0-9]),x}).x' | ||
| - 'main' | ||
| - 'next' | ||
| - 'next-major' | ||
| - 'beta' | ||
| - 'alpha' | ||
| pull_request: | ||
| types: [opened, synchronize] | ||
|
|
||
| jobs: | ||
| code_validation: | ||
| name: Code Validation | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v1 | ||
|
|
||
| - name: Use Node | ||
| uses: actions/setup-node@v2 | ||
| with: | ||
| node-version: '14' | ||
|
|
||
| - name: Install dependencies | ||
| uses: bahmutov/npm-install@v1 | ||
| with: | ||
| useLockFile: false | ||
|
|
||
| - name: Lint code | ||
| run: npm run lint | ||
|
|
||
| # TODO: reenable on v4 | ||
| # - name: Check format | ||
| # run: npm run format:check | ||
|
|
||
| test: | ||
| name: Tests (Node v${{ matrix.node }} - ESLint v${{ matrix.eslint }}) | ||
| runs-on: ubuntu-latest | ||
|
|
||
| strategy: | ||
| matrix: | ||
| node: ['10.12', '10', '12.0', '12', '14'] | ||
| eslint: [5, 6, 7] | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v1 | ||
|
|
||
| - name: Use Node | ||
| uses: actions/setup-node@v2 | ||
| with: | ||
| node-version: ${{ matrix.node }} | ||
|
|
||
| - name: Install dependencies | ||
| uses: bahmutov/npm-install@v1 | ||
| with: | ||
| useLockFile: false | ||
|
|
||
| - name: Install ESLint v${{ matrix.eslint }} | ||
| run: npm install --no-save eslint@${{ matrix.eslint }} | ||
|
|
||
| - name: Run tests | ||
| run: npm run test:ci | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,44 @@ | ||
| name: Release | ||
|
|
||
| on: | ||
| workflow_run: | ||
| workflows: ['CI'] | ||
| types: | ||
| - completed | ||
| push: | ||
| branches: | ||
| # semantic-release valid branches | ||
| - '+([0-9])?(.{+([0-9]),x}).x' | ||
| - 'main' | ||
| - 'next' | ||
| - 'next-major' | ||
| - 'beta' | ||
| - 'alpha' | ||
|
|
||
| jobs: | ||
| main: | ||
| name: NPM Release | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v1 | ||
|
|
||
| - name: Use Node | ||
| uses: actions/setup-node@v2 | ||
| with: | ||
| node-version: '14' | ||
|
|
||
| - name: Install dependencies | ||
| uses: bahmutov/npm-install@v1 | ||
| with: | ||
| useLockFile: false | ||
|
|
||
| - name: Build package | ||
| run: npm run build | ||
|
Comment on lines
+37
to
+38
There was a problem hiding this comment. should we build in There was a problem hiding this comment. Mmmm not really sure about that, what would be the purpose for it? I'm gonna merge both workflows into a single file actually, since it's triggering the release twice. There was a problem hiding this comment. to ensure there are no compilation errors (Typings for instance) committed without noticing? There was a problem hiding this comment. Ah! I'm doing that on v4 so on "Code Validation" job I'll add format check and typescript check steps. I'll create a PR later with those updates for v4! |
||
|
|
||
| - name: Release new version to NPM | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| NPM_TOKEN: ${{ secrets.NPM_TOKEN }} | ||
| run: npx semantic-release | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I think it is not necessary to use an extra GH action here, while a simple
should do the trick
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 npm-install action will cache stuff related to npm properly :)
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.
ohh nice