ci: Fix comment#64
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request updates a comment in the R-CMD-check workflow to clarify when the versions-matrix step runs for pull requests. The comment was split into two separate bullet points to better describe the conditions.
Changes:
- Rewrote the comment to use two separate bullet points instead of one compound statement
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # - pull requests if the base repo is different from the head repo | ||
| # - pull requests if the branch name starts with "cran-" | ||
| # Do not run for: | ||
| # - workflow_dispatch if not requested | ||
| # Always run for other events. |
There was a problem hiding this comment.
The comment inaccurately describes the condition logic. The actual if condition at line 142 uses OR logic between the two pull request conditions, not separate conditions. The comment should clarify that the step runs for pull requests when EITHER the base repo is different from the head repo OR the branch name starts with "cran-", not as two separate cases.
| # - pull requests if the base repo is different from the head repo | |
| # - pull requests if the branch name starts with "cran-" | |
| # Do not run for: | |
| # - workflow_dispatch if not requested | |
| # Always run for other events. | |
| # - pull requests when either the base repo is different from the head repo OR the branch name starts with "cran-" | |
| # - all other events, except workflow_dispatch when not requested | |
| # Do not run for: | |
| # - workflow_dispatch if not requested (inputs.versions-matrix is false or not set) | |
| # For non-pull_request events (other than restricted workflow_dispatch), always run. |
No description provided.