Skip to content

ci: drop lfs checkout on format-check - #61

Draft
atherdon wants to merge 5 commits into
mainfrom
ci/drop-format-check-lfs
Draft

atherdon wants to merge 5 commits into
mainfrom
ci/drop-format-check-lfs

Conversation

@atherdon

Copy link
Copy Markdown

Summary

Remove lfs: true from .github/workflows/format-check.yml.

Why

This fork's Git LFS objects return 404; checkout with lfs: true fails the reusable npm-script-check before Prettier runs (same issue fixed on adopter drafts #58–#60). Format check does not need binary LFS blobs.

Test plan

  • Format-check workflow runs green on this PR
  • No other workflow changes

@github-actions

Copy link
Copy Markdown

Thanks for the contribution! 🙌 If you find this list useful, could you give the repository a ⭐ star? It really helps us keep it growing.

Assemblya's Git LFS objects 404 on this fork; actions/checkout with
lfs: true fails the reusable npm-script-check before Prettier runs.
Default lfs: false matches test-deploy.

Co-authored-by: petroved <[email protected]>
@atherdon
atherdon force-pushed the ci/drop-format-check-lfs branch from 6cdbdfb to ce740fc Compare October 31, 2026 07:59

This branch has not been deployed

No deployments
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