Skip to content

Commit

Permalink
Update Codecov
Browse files Browse the repository at this point in the history
  • Loading branch information
ehmicky committed Apr 28, 2024
1 parent 400890c commit 8c4e7cb
Show file tree
Hide file tree
Showing 7 changed files with 6 additions and 73 deletions.
6 changes: 5 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ name: Build
on: workflow_call
jobs:
combinations:
permissions:
id-token: write
name: ${{ matrix.os }}, Node.js ${{matrix.node}}
# git push --tags does not create any commits so should not trigger a new
# workflow. There are no ways to skip workflows, so we skip jobs instead
Expand Down Expand Up @@ -39,4 +41,6 @@ jobs:
- name: Tests
run: npx gulp unit
- name: Upload test coverage
run: npx gulp uploadCoverage
uses: codecov/codecov-action@v4
with:
use_oidc: true
5 changes: 0 additions & 5 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -152,11 +152,6 @@ Release a new patch/minor/major version on npm and GitHub using

Only the repository owner can perform these tasks.

## `gulp uploadCoverage`

In CI ([GitHub actions](https://github.com/features/actions)), upload test
coverage (produced by `c8 ava`) to [codecov](https://codecov.io/).

## `gulp checkCoverage`

Ensures that files are fully covered by tests using
Expand Down
21 changes: 0 additions & 21 deletions gulp/download.js

This file was deleted.

1 change: 0 additions & 1 deletion gulpfile.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,3 @@
import './src/tasks/builder/register.js'

export * from './src/main.js'
export { download } from './gulp/download.js'
2 changes: 1 addition & 1 deletion src/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ export { build, buildWatch } from './tasks/builder/main.js'
export { check, checkWatch } from './tasks/check/main.js'
export { unit, unitWatch } from './tasks/unit/main.js'
export { type, typeWatch } from './tasks/type.js'
export { uploadCoverage, checkCoverage } from './tasks/cover/main.js'
export { checkCoverage } from './tasks/cover/main.js'
export { warn, warnWatch } from './tasks/warn.js'
export {
releaseMajor,
Expand Down
6 changes: 0 additions & 6 deletions src/tasks/cover/main.js
Original file line number Diff line number Diff line change
@@ -1,13 +1,7 @@
import { checkCoverage } from './check.js'
import { uploadCoverage } from './upload.js'

// eslint-disable-next-line fp/no-mutation
uploadCoverage.description = 'Upload test coverage to CI'

// eslint-disable-next-line fp/no-mutation
checkCoverage.description = 'Ensure source files are fully covered by tests'

// eslint-disable-next-line no-duplicate-imports
export { uploadCoverage } from './upload.js'
// eslint-disable-next-line no-duplicate-imports
export { checkCoverage } from './check.js'
38 changes: 0 additions & 38 deletions src/tasks/cover/upload.js

This file was deleted.

0 comments on commit 8c4e7cb

Please sign in to comment.