Call the shared CTTIR workflows - #2
Merged
Merged
Conversation
Replaces copied R-CMD-check and test-coverage YAML with caller stubs and adds the house lint workflow. Behaviour that comes with the shared versions: an oldrel-3 job so the R version DESCRIPTION claims is actually built, --as-cran, a coverage gate computed with covr and an explicit exit rather than delegated to Codecov, and fail_ci_if_error on upload. Coverage floor is 0 for now - this fixes the measurement, not the gate. Lint is report-only.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Converts this package to the shared CTTIR workflows (CTTIR/.github#2), validated by the hexmakR pilot (CTTIR/hexmakR#3).
What changes
R-CMD-checkgains anoldrel-3job, so the R versionDESCRIPTIONactually claims to support is built, and passes--as-cran.test-coveragecomputes its gate in the workflow with covr and an explicit exit instead of delegating to Codecov, and uploads withfail_ci_if_error: true. Nine of 22 repos currently reportactive: false, totals: nullto Codecov behind a green build, because a deactivated Codecov fails nothing.lintis new, running the CTTIR house naming linters. Report-only — turning it on as a gate would red five repositories on day one.Deliberately conservative
floor: 0on coverage, so this changes the measurement without adding a gate. Set a real floor once each package is re-baselined — the old configs usedtarget: auto, threshold: 1%, which compares only against the base commit and so permits unlimited decay in 1% steps.pkgdown,api-lockand any repo-specific workflows are untouched.