Skip to content

Commit

Permalink
ci: clean up to be ready for master
Browse files Browse the repository at this point in the history
  • Loading branch information
michaelfig committed May 20, 2021
1 parent 856abbe commit 464b464
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/test-all-packages.yml
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ jobs:
strategy:
matrix:
node-version: ['14.x']
# TODO: if: ${{github.event_name == 'push' && github.ref == 'refs/heads/master'}}
if: ${{github.event_name == 'push' && github.ref == 'refs/heads/master'}}
steps:
- uses: actions/setup-node@v1
with:
Expand Down Expand Up @@ -162,7 +162,7 @@ jobs:
- uses: nwtgck/actions-netlify@v1.1
with:
# Production deployment if a commit to master.
production-deploy: ${{ github.ref != 'TODOrefs/heads/master' }}
production-deploy: ${{ github.ref == 'refs/heads/master' }}
publish-dir: coverage/html
# SECURITY: we don't want to hand out the Github token to this action.
# github-token: ${{ secrets.GITHUB_TOKEN }}
Expand Down
4 changes: 2 additions & 2 deletions COVERAGE.md
Original file line number Diff line number Diff line change
Expand Up @@ -42,13 +42,13 @@ it also support nesm by:

2. Make the following changes to its `package.json` (omitting comments):

```json
```js
{
// Enable nesm support.
"type": "module",
"scripts": {
// The following line enables coverage generation from the top.
"test:c8": "c8 $C8_OPTIONS ava --config=ava-nesm.config.js",
"test:c8": "c8 $C8_OPTIONS ava --config=ava-nesm.config.js"
}
}
```
Expand Down
3 changes: 2 additions & 1 deletion packages/eslint-config/eslint-config.json
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,7 @@
"**/output/**",
"bundles/**",
"dist/**",
"test262/**"
"test262/**",
"ava*.config.js"
]
}

0 comments on commit 464b464

Please sign in to comment.