Skip to content

WEBDEV-8971: Run lint in CI and fix prettier on main - #79

Open
jbuckner wants to merge 1 commit into
mainfrom
WEBDEV-8971-lint-in-ci
Open

WEBDEV-8971: Run lint in CI and fix prettier on main#79
jbuckner wants to merge 1 commit into
mainfrom
WEBDEV-8971-lint-in-ci

Conversation

@jbuckner

Copy link
Copy Markdown
Collaborator

CI never ran pnpm run lint, so main sat failing prettier on demo/story-template.test.ts. This adds a lint job to ci.yml, fixes the formatting, and ignores .claude/ and .wireit/ in eslint so local junk doesn't fail the lint (prettier already skips them via .gitignore).

https://webarchive.jira.com/browse/WEBDEV-8971

🤖 Generated with Claude Code

https://claude.ai/code/session_01CUM1e29MLyK3kBDiZYgERx

@github-actions

Copy link
Copy Markdown
PR Preview Action v1.8.1

🚀 View preview at
https://internetarchive.github.io/elements/pr/pr-79/

Built to branch ghpages at 2026-08-26 23:47 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.12%. Comparing base (8c054a6) to head (1f621b9).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #79   +/-   ##
=======================================
  Coverage   81.12%   81.12%           
=======================================
  Files          27       27           
  Lines         927      927           
  Branches      227      227           
=======================================
  Hits          752      752           
  Misses        113      113           
  Partials       62       62           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

3 participants