-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document how to do custom system installs #23
Comments
You deleted the section I wrote on that topic from the README :P Advanced UsageIf you want to customize your workflow in a way not possible with the inputs provided, you can use the container supplied by this repository as a base for a custom workflow. A simple example using the name: pkgcheck
concurrency:
group: ${{ github.workflow }}-${{ github.head_ref }}
cancel-in-progress: true
on:
# Manual trigger
workflow_dispatch:
# Run on every push to main
push:
branches:
- main
# Run on every PR to main
pull_request:
branches:
- main
jobs:
check:
runs-on: ubuntu-latest
container: ghcr.io/ropensci-review-tools/pkgcheck-action:latest
env:
GITHUB_PAT: ${{ github.token }}
steps:
- uses: actions/checkout@v2
- name: Run pkgcheck
id: pkgcheck
run: source("check.R") # adjust path to script here
shell: Rscript {0}
- uses: actions/upload-artifact@v2
with:
name: visual-network
path: "${{ steps.pkgcheck.outputs.visnet_path }}"
- uses: actions/upload-artifact@v2
with:
name: results
path: "${{ steps.pkgcheck.outputs.results }}"
|
But that merely uses the container's entrypoint, but doesn't modify the container in any way. What would be needed would be a way to modify the actual container via |
They could either use If they want to prepare their own container they can just Edit: A small correction, using the above workflow does not call the |
This could also be used to cache dependencies for packages with large, complex dependencies. |
@assignUser I don't even know if this is possible with the current docker-based composite action, but there'll definitely be a need for people to insert their own custom system installation steps. Thoughts?
The text was updated successfully, but these errors were encountered: