Skip to content

chore: surface prettier formatting warnings in lint:misc - #1114

Merged
rekmarks-consensys-1 merged 1 commit into
mainfrom
fix/prettier-log-level-warn
Sep 18, 2026
Merged

rekmarks-consensys-1 merged 1 commit into
mainfrom
fix/prettier-log-level-warn

Conversation

@rekmarks-consensys-1

@rekmarks-consensys-1 rekmarks-consensys-1 commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

--log-level error suppresses prettier's per-file formatting diagnostics, which are emitted at the warn level. This means yarn lint:misc --check would fail with a non-zero exit code but no output indicating which files were affected.

Changes

  • package.json: lint:misc--log-level error--log-level warn

Testing

No automated tests needed. Verified by running yarn lint:misc --check against a file with a formatting issue to confirm the diagnostic message is now visible.

🤖 Generated with Claude Code


Note

Low Risk
Single dev-tool logging tweak in root package.json with no runtime or security impact.

Overview
Updates the root lint:misc Prettier invocation so formatting failures are visible when lint runs with --check (via yarn lint).

--log-level changes from error to warn because Prettier reports per-file formatting issues at the warn level; with error, yarn lint:misc --check could exit non-zero without printing which files were wrong.

Reviewed by Cursor Bugbot for commit 83852db. Bugbot is set up for automated code reviews on this repo. Configure here.

@rekmarks-consensys-1
rekmarks-consensys-1 requested a review from a team as a code owner September 18, 2026 07:38
@rekmarks-consensys-1 rekmarks-consensys-1 added the no-changelog Indicates that no changelog updates are required, and that related CI checks should be skipped. label Sep 18, 2026
@rekmarks-consensys-1
rekmarks-consensys-1 marked this pull request as draft September 18, 2026 07:39
@rekmarks-consensys-1
rekmarks-consensys-1 changed the base branch from main to rekmarks-consensys-1-patch-1 September 18, 2026 07:39
Base automatically changed from rekmarks-consensys-1-patch-1 to main September 18, 2026 16:52
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@rekmarks-consensys-1
rekmarks-consensys-1 marked this pull request as ready for review September 18, 2026 16:56
@rekmarks-consensys-1 rekmarks-consensys-1 changed the title fix: surface prettier formatting warnings in lint:misc chore: surface prettier formatting warnings in lint:misc Sep 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 73.04%
⬇️ -0.01%
9789 / 13401
🔵 Statements 72.89%
⬇️ -0.01%
9953 / 13653
🔵 Functions 73.6%
🟰 ±0%
2301 / 3126
🔵 Branches 67.25%
⬇️ -0.02%
4022 / 5980
File CoverageNo changed files found.
Generated in workflow #4836 for commit 83852db by the Vitest Coverage Report Action

@rekmarks-consensys-1
rekmarks-consensys-1 added this pull request to the merge queue Sep 18, 2026
Merged via the queue into main with commit 93936c4 Sep 18, 2026
30 checks passed
@rekmarks-consensys-1
rekmarks-consensys-1 deleted the fix/prettier-log-level-warn branch September 18, 2026 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that no changelog updates are required, and that related CI checks should be skipped.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants