Skip to content

Update CLAUDE.md 80-char line rule, as suggested by Opus 4.6#335

Merged
padelsbach merged 1 commit intowolfSSL:masterfrom
cconlon:claudeMd80Chars
Feb 24, 2026
Merged

Update CLAUDE.md 80-char line rule, as suggested by Opus 4.6#335
padelsbach merged 1 commit intowolfSSL:masterfrom
cconlon:claudeMd80Chars

Conversation

@cconlon
Copy link
Copy Markdown
Member

@cconlon cconlon commented Feb 23, 2026

Opus 4.5 followed the previous 80-character line wrapping nicely. But, Opus 4.6 seems to wrap too early. Claude Opus 4.6 suggested the following change, submitting here to try out.

Copilot AI review requested due to automatic review settings February 23, 2026 22:59
@cconlon cconlon self-assigned this Feb 23, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s contributor/style guidance in CLAUDE.md to clarify how the 80-character line limit should be applied when wrapping text.

Changes:

  • Rewords the 80-character rule to specify wrapping only when a line would exceed 80 characters.

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

Comment thread CLAUDE.md
Copy link
Copy Markdown
Contributor

@padelsbach padelsbach left a comment

Choose a reason for hiding this comment

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

Looks good, thanks

@cconlon cconlon assigned padelsbach and unassigned cconlon Feb 24, 2026
@padelsbach padelsbach merged commit b937eec into wolfSSL:master Feb 24, 2026
101 checks passed
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