Skip to content

Fill in cran-comments.md test environments - #7

Merged
agallinat merged 1 commit into
mainfrom
cran-comments-envs
Oct 3, 2026
Merged

agallinat merged 1 commit into
mainfrom
cran-comments-envs

Conversation

@agallinat

Copy link
Copy Markdown
Owner

All five GitHub Actions platforms (ubuntu devel/release/oldrel-1, macOS
release, windows release) have now passed repeatedly, so they can be listed
honestly. The earlier TODO existed because the workflow had never run when
the file was written; listing them then would have asserted results that
did not exist.

Also noted that the CI jobs use error_on = "warning", so a warning fails
them as well as an error -- which is stricter than a plain --as-cran run
and worth telling a reviewer.

Recorded the one way the file does not currently match reality: while
DESCRIPTION carries the .9000 development version, the
incoming-feasibility note additionally reports "Version contains large
components". That vanishes once a release version is set, which is the
state the file describes, so the mismatch is expected rather than stale --
but it would be confusing to find later without explanation.

Co-Authored-By: Claude Opus 5 (1M context) noreply@anthropic.com

All five GitHub Actions platforms (ubuntu devel/release/oldrel-1, macOS
release, windows release) have now passed repeatedly, so they can be listed
honestly. The earlier TODO existed because the workflow had never run when
the file was written; listing them then would have asserted results that
did not exist.

Also noted that the CI jobs use error_on = "warning", so a warning fails
them as well as an error -- which is stricter than a plain --as-cran run
and worth telling a reviewer.

Recorded the one way the file does not currently match reality: while
DESCRIPTION carries the .9000 development version, the
incoming-feasibility note additionally reports "Version contains large
components". That vanishes once a release version is set, which is the
state the file describes, so the mismatch is expected rather than stale --
but it would be confusing to find later without explanation.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@agallinat
agallinat merged commit 0a82c54 into main Oct 3, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant