Skip to content

Commit 07dfa35

Browse files
authored
enable coverage on PRs again (#152)
1 parent 43156e3 commit 07dfa35

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

.github/workflows/test-coverage.yaml

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,16 +3,18 @@ on:
33
push:
44
branches:
55
- main
6-
# disabled for PRs, because OpenCage key is not available there
7-
# pull_request:
8-
# branches:
9-
# - main
6+
pull_request:
7+
branches:
8+
- main
109

1110
name: test-coverage
1211

1312
jobs:
1413
test-coverage:
1514
runs-on: ubuntu-latest
15+
# skip for forks as secrets are only available in main repo
16+
# keep an eye on https://github.com/orgs/community/discussions/9098
17+
if: ${{ github.repository == 'ropensci/opencage' }}
1618
env:
1719
GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}
1820

NEWS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
## Internals
1111

1212
* {opencage} now uses [{testthat} 3e](https://testthat.r-lib.org/articles/third-edition.html) for unit tests ([#141](https://github.com/ropensci/opencage/issues/141)).
13-
* GitHub actions workflows updated ([#142](https://github.com/ropensci/opencage/issues/142), [#149](https://github.com/ropensci/opencage/pull/149)).
13+
* GitHub actions workflows updated ([#142](https://github.com/ropensci/opencage/issues/142), [#149](https://github.com/ropensci/opencage/pull/149), [#152](https://github.com/ropensci/opencage/pull/152)).
1414
* Use [{lintr} version 3.0](https://www.tidyverse.org/blog/2022/07/lintr-3-0-0/) and add "package development" linters ([#144](https://github.com/ropensci/opencage/pull/144)).
1515
* `countrycodes` source and script were moved to `data-raw` ([#146](https://github.com/ropensci/opencage/pull/146)).
1616
* Add CITATION.cff and a corresponding GitHub action ([#148](https://github.com/ropensci/opencage/pull/148)).

0 commit comments

Comments
 (0)