Skip to content

Switch to shared checks.yml for pre-commit, zizmor, lychee#116

Draft
bschwedler wants to merge 4 commits into
mainfrom
shared-checks-workflow
Draft

Switch to shared checks.yml for pre-commit, zizmor, lychee#116
bschwedler wants to merge 4 commits into
mainfrom
shared-checks-workflow

Conversation

@bschwedler

Copy link
Copy Markdown
Contributor

Replaces the inline lint and zizmor jobs with a checks caller
that delegates to the shared checks.yml in posit-dev/images-shared.

Depends on:

The final commit is a REVERTME that pins the workflow to the
worktree-lychee-link-checking branch for testing. It must be
reverted before this PR merges, once #625 is merged to main.

Replaces the inline lint and zizmor jobs with a single checks caller
that delegates to posit-dev/images-shared. Pre-commit, zizmor, and
lychee link checking are all included via the shared workflow.
Points at the worktree-lychee-link-checking branch instead of main
to test the workflow before it merges.

Revert this commit once posit-dev/images-shared#625 is merged.
The configuration appendix URL was missing the .html extension.
.lycheeignore excludes the fixed URL and support.posit.co, both
of which fail lychee with 404/403 in CI.
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