Skip to content

Commit

Permalink
fix: fix extensions, cov,formating, types, errors and workers (#168)
Browse files Browse the repository at this point in the history
- fixes extension waiting for tests
  - use polling because RaF doesn't work in extension
- code coverage for undefined src line.
- normalise code formatting
- add types
- improved error handling
- better worker script loading

BREAKING CHANGE: too many changes and esbuild updated
  • Loading branch information
hugomrdias authored Apr 5, 2021
1 parent 6938ebd commit 9593c3d
Show file tree
Hide file tree
Showing 39 changed files with 5,480 additions and 4,636 deletions.
18 changes: 10 additions & 8 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,14 @@ about: Create a report to help us improve
title: ''
labels: ''
assignees: ''

---

**Describe the bug**
A clear and concise description of what the bug is.

**To Reproduce**
Steps to reproduce the behavior:

1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
Expand All @@ -24,15 +24,17 @@ A clear and concise description of what you expected to happen.
If applicable, add screenshots to help explain your problem.

**Desktop (please complete the following information):**
- OS: [e.g. iOS]
- Browser [e.g. chrome, safari]
- Version [e.g. 22]

- OS: [e.g. iOS]
- Browser [e.g. chrome, safari]
- Version [e.g. 22]

**Smartphone (please complete the following information):**
- Device: [e.g. iPhone6]
- OS: [e.g. iOS8.1]
- Browser [e.g. stock browser, safari]
- Version [e.g. 22]

- Device: [e.g. iPhone6]
- OS: [e.g. iOS8.1]
- Browser [e.g. stock browser, safari]
- Version [e.g. 22]

**Additional context**
Add any other context about the problem here.
1 change: 0 additions & 1 deletion .github/ISSUE_TEMPLATE/feature_request.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ about: Suggest an idea for this project
title: ''
labels: ''
assignees: ''

---

**Is your feature request related to a problem? Please describe.**
Expand Down
70 changes: 48 additions & 22 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,39 +4,65 @@ env:
FORCE_COLOR: 1
on:
push:
branches: [ master ]
branches: [master]
pull_request:
branches: [ master ]
branches: [master]

jobs:
linux:
name: "Linux"
name: 'Linux'
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: microsoft/playwright-github-action@v1
- run: npm install
- run: npm run lint
- run: xvfb-run -a npm run test
- run: ./cli.js mocks/test.mocha.js --browser firefox
- run: ./cli.js mocks/test.mocha.js --browser webkit
- uses: actions/checkout@v2
- uses: microsoft/playwright-github-action@v1
- run: npm install
- run: npm run lint
- run: xvfb-run -a npm run test
- run: ./cli.js mocks/test.mocha.js --browser firefox
- run: ./cli.js mocks/test.mocha.js --browser webkit

macos:
name: "Mac"
name: 'Mac'
runs-on: macos-latest
steps:
- uses: actions/checkout@v2
- run: npm install
- run: npm test
- run: ./cli.js mocks/test.mocha.js --browser firefox
- run: ./cli.js mocks/test.mocha.js --browser webkit
- uses: actions/checkout@v2
- run: npm install
- run: npm test
- run: ./cli.js mocks/test.mocha.js --browser firefox
- run: ./cli.js mocks/test.mocha.js --browser webkit

win:
name: "Win"
name: 'Win'
runs-on: windows-latest
steps:
- uses: actions/checkout@v2
- run: npm install
- run: npm test
- run: ./cli.js mocks/test.mocha.js --browser firefox
- run: ./cli.js mocks/test.mocha.js --browser webkit
- uses: actions/checkout@v2
- run: npm install
- run: npm test
- run: ./cli.js mocks/test.mocha.js --browser firefox
- run: ./cli.js mocks/test.mocha.js --browser webkit
release:
runs-on: ubuntu-latest
needs:
- linux
- macos
- win
if: github.event_name == 'push' && github.ref == 'refs/heads/master'
steps:
- uses: GoogleCloudPlatform/release-please-action@v2
id: release
with:
token: ${{ secrets.GITHUB_TOKEN }}
release-type: node
- uses: actions/checkout@v2
if: ${{ steps.release.outputs.release_created }}
- uses: actions/setup-node@v2
with:
node-version: 14
registry-url: 'https://registry.npmjs.org'
if: ${{ steps.release.outputs.release_created }}
- uses: bahmutov/npm-install@v1
if: ${{ steps.release.outputs.release_created }}
- run: npm publish
env:
NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}
if: ${{ steps.release.outputs.release_created }}
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -13,3 +13,4 @@ node_modules
coverage.lcov
yarn-error.log
yarn.lock
dist
Loading

0 comments on commit 9593c3d

Please sign in to comment.