Skip to content

Conversation

@chr-tatu
Copy link
Contributor

@chr-tatu chr-tatu commented Dec 31, 2025

Summary

Disables automatic line wrapping in Prettier by setting proseWrap: "never" in package.json.

Changes

  • Changed proseWrap from "always" to "never" in the prettier configuration
  • All other file changes were from the pre-commit hook

This prevents the pre-commit hook from automatically wrapping lines in markdown files while maintaining other formatting rules for code.

@chr-tatu chr-tatu marked this pull request as ready for review December 31, 2025 13:35
Copy link
Contributor

@rabidpaul rabidpaul left a comment

Choose a reason for hiding this comment

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

I support this. I don't think most of these whitespace manipulation adds more diff confusion than it creates readability, especially in a markdown repo where people read in previews that ignore such breaks.

@@ -1,82 +1,82 @@
name,ring,quadrant,isNew,description
AI Code Assistants,assess,techniques,TRUE, "Tools like GitHub Copilot, Claude to assist with code generation and suggestions"
Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't parse all of this but it looks like most of the changes are just sorting.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are from the pre-commit hook.

Copy link
Contributor

@joeyAghion joeyAghion left a comment

Choose a reason for hiding this comment

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

👍 from me. So confusing otherwise.

@chr-tatu chr-tatu merged commit 8239141 into main Jan 5, 2026
2 checks passed
@chr-tatu chr-tatu deleted the chr-tatu/remove-auto-line-wrap branch January 5, 2026 14:13
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.

4 participants