Update README.md#3683
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
🚩 PR has no meaningful content change
This PR adds only two trailing blank lines to README.md with no accompanying description or purpose. Per the REVIEW.md "Scope discipline" guideline, changes should "do one thing well." This change doesn't appear to accomplish anything — it modifies no content, fixes no typo, and adds no documentation. It may have been an accidental commit (e.g., an editor auto-inserting newlines). The author should clarify intent or close the PR.
Was this helpful? React with 👍 or 👎 to provide feedback.
|
|
||
|
|
There was a problem hiding this comment.
🟡 Extra trailing blank lines violate pre-commit end-of-file-fixer hook
The PR adds two blank lines at the end of README.md, resulting in the file ending with multiple trailing newlines. The repository's .pre-commit-config.yaml:12 configures the end-of-file-fixer hook (from pre-commit/pre-commit-hooks), which enforces that files end with exactly one newline. This change would be reverted by that hook, indicating it's a formatting violation. Per REVIEW.md's mandatory "Formatting and linting clean" check, this should be flagged.
Was this helpful? React with 👍 or 👎 to provide feedback.
Summary
Rationale
How has this been tested?