Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add support for multiple triggers? #135

Open
iantaylor-NOAA opened this issue Jul 26, 2024 · 1 comment
Open

add support for multiple triggers? #135

iantaylor-NOAA opened this issue Jul 26, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@iantaylor-NOAA
Copy link
Collaborator

It might be helpful to run doc-and-style, for instance on both pushes to the main branch and also pull requests to resolve issues before merging. Currently the build_trigger argument only allows a single value from the list, but couldn't it be modified to use match.arg(..., several.ok = TRUE) to allow multiple options? Other changes in the code would presumably be required as well.

build_trigger <- match.arg(
arg = build_trigger,
choices = c(
"push_to_main", "pull_request", "manually",
"weekly"
)
)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants