Skip to content

Commit

Permalink
Remove lookup-refs (#365)
Browse files Browse the repository at this point in the history
Part of insightsengineering/coredev-tasks#609

From now on, we will provide development dependencies in 
```
Remotes: repo/project@branch
```
format, so it's explicitly visible in the DESCRIPTION file and can be
handled by `pak::install`, `renv::install` and `remotes::install`.

With development dependencies specified in CJ Pipelines configuration,
this connection was hidden, and it was hard to install the package from
the main branch (or any other branch) locally from user's machine.
  • Loading branch information
m7pr authored Jan 31, 2025
1 parent c0de671 commit 2f299e8
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 24 deletions.
8 changes: 0 additions & 8 deletions .github/workflows/check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,6 @@ jobs:
unit-test-report-brand: >-
https://raw.githubusercontent.com/insightsengineering/hex-stickers/main/thumbs/teal.png
deps-installation-method: setup-r-dependencies
lookup-refs: |
insightsengineering/teal.code

r-cmd-non-cran:
name: R CMD Check (non-CRAN) 🧬
Expand All @@ -66,8 +64,6 @@ jobs:
checking for unstated dependencies in vignettes .* NOTE
checking top-level files .* NOTE
deps-installation-method: setup-r-dependencies
lookup-refs: |
insightsengineering/teal.code

coverage:
name: Coverage πŸ“”
Expand All @@ -78,8 +74,6 @@ jobs:
additional-env-vars: |
NOT_CRAN=true
deps-installation-method: setup-r-dependencies
lookup-refs: |
insightsengineering/teal.code
linter:
if: github.event_name != 'push'
name: SuperLinter πŸ¦Έβ€β™€οΈ
Expand All @@ -92,8 +86,6 @@ jobs:
with:
auto-update: true
deps-installation-method: setup-r-dependencies
lookup-refs: |
insightsengineering/teal.code
gitleaks:
name: gitleaks πŸ’§
uses: insightsengineering/r.pkg.template/.github/workflows/gitleaks.yaml@main
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/docs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,5 +43,3 @@ jobs:
default-landing-page: latest-tag
additional-unit-test-report-directories: unit-test-report-non-cran
deps-installation-method: setup-r-dependencies
lookup-refs: |
insightsengineering/teal.code
8 changes: 0 additions & 8 deletions .github/workflows/release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,6 @@ jobs:
with:
default-landing-page: latest-tag
deps-installation-method: setup-r-dependencies
lookup-refs: |
insightsengineering/teal.code
validation:
name: R Package Validation report πŸ“ƒ
needs: release
Expand All @@ -27,8 +25,6 @@ jobs:
REPO_GITHUB_TOKEN: ${{ secrets.REPO_GITHUB_TOKEN }}
with:
deps-installation-method: setup-r-dependencies
lookup-refs: |
insightsengineering/teal.code
release:
name: Create release πŸŽ‰
uses: insightsengineering/r.pkg.template/.github/workflows/release.yaml@main
Expand All @@ -54,8 +50,6 @@ jobs:
unit-test-report-brand: >-
https://raw.githubusercontent.com/insightsengineering/hex-stickers/main/thumbs/teal.png
deps-installation-method: setup-r-dependencies
lookup-refs: |
insightsengineering/teal.code
coverage:
name: Coverage πŸ“”
needs: [release, docs]
Expand All @@ -66,8 +60,6 @@ jobs:
additional-env-vars: |
NOT_CRAN=true
deps-installation-method: setup-r-dependencies
lookup-refs: |
insightsengineering/teal.code
wasm:
name: Build WASM packages πŸ§‘β€πŸ­
needs: release
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/scheduled.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,6 @@ jobs:
)
name: revdepcheck ↩️
uses: insightsengineering/r.pkg.template/.github/workflows/revdepcheck.yaml@main
with:
lookup-refs: |
insightsengineering/teal.code
rhub:
if: >
github.event_name == 'schedule' || (
Expand All @@ -67,6 +64,3 @@ jobs:
)
name: R-hub 🌐
uses: insightsengineering/r.pkg.template/.github/workflows/rhub.yaml@main
with:
lookup-refs: |
insightsengineering/teal.code

0 comments on commit 2f299e8

Please sign in to comment.