Skip to content

Conversation

@simonLeary42
Copy link
Collaborator

No description provided.

This comment was marked as outdated.

@simonLeary42 simonLeary42 requested a review from Copilot September 9, 2025 20:24

This comment was marked as outdated.

@simonLeary42 simonLeary42 requested a review from Copilot September 9, 2025 20:28

This comment was marked as outdated.

@simonLeary42 simonLeary42 requested a review from Copilot September 9, 2025 20:32
Copy link
Contributor

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

This PR enforces a maximum line length of 100 characters instead of PSR-12's default 120 characters across the PHP codebase. The changes involve refactoring long lines by breaking them into multiple lines with proper indentation while maintaining code functionality.

Key changes:

  • Configuration update to enforce 100-character line limit via custom PHP_CodeSniffer ruleset
  • Systematic refactoring of long lines across multiple PHP files using appropriate line breaks
  • Documentation update to reflect the new line length requirement

Reviewed Changes

Copilot reviewed 30 out of 31 changed files in this pull request and generated 5 comments.

File Description
.phpcs-ruleset.xml New custom ruleset defining 100-character line limit
.pre-commit-config.yaml Updated to use custom ruleset instead of PSR12
CONTRIBUTING.md Added documentation about 100-character line limit
Multiple PHP files Refactored long lines to comply with 100-character limit

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@simonLeary42 simonLeary42 merged commit e815a78 into main Sep 9, 2025
3 checks passed
@simonLeary42 simonLeary42 deleted the line-len-100 branch September 9, 2025 20:37
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.

2 participants