Skip to content

perf(eslint): enable caching for lint scripts and CI#227

Merged
jhnns merged 2 commits into
mainfrom
perf/eslint-cache
Jun 28, 2026
Merged

perf(eslint): enable caching for lint scripts and CI#227
jhnns merged 2 commits into
mainfrom
perf/eslint-cache

Conversation

@jhnns

@jhnns jhnns commented Jun 27, 2026

Copy link
Copy Markdown
Member

Summary

  • Add --cache to every eslint CLI invocation in npm test scripts
  • Git-ignore .eslintcache files locally while caching **/.eslintcache in CI
  • Refresh the GitHub Actions eslint cache each run via restore-keys so subdirectory fixture caches are reused

Test plan

  • Verified eslint --cache creates .eslintcache locally
  • Verified .eslintcache is ignored by git in root and fixture test dirs
  • CI passes on this PR

Add --cache to every eslint test script, gitignore .eslintcache files,
and restore/update eslint cache across CI runs including fixture subdirs.
Copilot AI review requested due to automatic review settings June 27, 2026 15:45
@pkg-pr-new

pkg-pr-new Bot commented Jun 27, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@peerigon/configs@227

commit: 3e07324

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves linting performance in @peerigon/configs by enabling ESLint’s on-disk cache across local npm scripts and GitHub Actions, with CI configured to persist .eslintcache between runs.

Changes:

  • Add --cache to all npm scripts that invoke eslint.
  • Ignore .eslintcache files via .gitignore so local cache artifacts aren’t committed.
  • Cache **/.eslintcache in GitHub Actions to reuse lint results across CI runs.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
package.json Enables ESLint caching for all lint-related npm test scripts.
.gitignore Ignores .eslintcache files to prevent committing generated cache artifacts.
.github/workflows/test-and-release.yml Caches .eslintcache files across CI runs (including fixture subdirectories).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/test-and-release.yml Outdated
@jhnns jhnns merged commit 9bcba40 into main Jun 28, 2026
2 checks passed
@jhnns jhnns deleted the perf/eslint-cache branch June 28, 2026 12:31
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.

2 participants