ci: Fix comment - #824
Conversation
There was a problem hiding this comment.
Pull request overview
This PR attempts to fix a comment in the R-CMD-check GitHub Actions workflow that describes when the versions-matrix step should run. The original comment incorrectly stated that the step should NOT run when the branch name starts with "cran-", but the actual code logic shows it SHOULD run in that case.
Changes:
- Updated comment to correctly indicate that the step runs for pull requests when the branch name starts with "cran-"
💡 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-" |
There was a problem hiding this comment.
The comment is still inaccurate. The actual logic on line 142 uses OR conditions, not separate scenarios. For pull requests, the step runs if EITHER the repos are different OR the branch name starts with "cran-", not both conditions separately. Consider revising to: "- pull requests if the base repo is different from the head repo OR the branch name starts with 'cran-'"
| # - pull requests if the base repo is different from the head repo | |
| # - pull requests if the branch name starts with "cran-" | |
| # - pull requests if the base repo is different from the head repo OR the branch name starts with "cran-" |
No description provided.