Updated own reference to the standard and fixed violations.

[AlexSkrypnyk]

Summary by CodeRabbit

• Chores

  • Consolidated code quality configuration by replacing multiple specific naming convention rules with a broader aggregated rule.

• Tests

  • Updated test data provider naming conventions for consistency and clarity across the test suite.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The changes consolidate two separate PHPCS naming convention rules (LocalVariableSnakeCase and ParameterSnakeCase) into a single aggregated DrevOps rule in the phpcs.xml configuration, and rename data provider methods across three test files to follow a consistent naming convention pattern.

Changes

Cohort / File(s)	Change Summary
PHPCS Configuration phpcs.xml	Replaces two specific DrevOps naming rules (DrevOps.NamingConventions.LocalVariableSnakeCase and DrevOps.NamingConventions.ParameterSnakeCase) with a single aggregated rule reference (DrevOps).
Test Data Provider Renaming tests/Unit/AbstractVariableSnakeCaseSniffTest.php, tests/Unit/LocalVariableSnakeCaseSniffTest.php, tests/Unit/ParameterSnakeCaseSniffTest.php	Renames data provider methods to follow consistent naming convention: providerSnakeCase → dataProviderSnakeCaseDetection, and providerProcessCases → dataProviderProcess. Updates corresponding DataProvider attributes to reference the new method names.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• All changes follow consistent patterns (rule consolidation + repetitive method renames across test files)
• No logic modifications or behavioral changes
• Straightforward configuration and naming alignment updates

Possibly related PRs

• Separated rules into LocalVariableSnakeCase and ParameterSnakeCase. #2: Relates to the same snake_case naming rule configuration; this PR consolidates the rules while the referenced PR split a single rule into separate LocalVariableSnakeCase and ParameterSnakeCase sniffs.
• Fixed snake_case not being updated in the docblocks. #8: Touches the ParameterSnakeCase test files and rule configuration that are modified in this PR.

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'Updated own reference to the standard and fixed violations' is vague and does not clearly describe the specific changes made in the pull request.	Use a more specific title that clearly describes the main change, such as 'Consolidate DrevOps PHPCS rules and update test data providers' or 'Refactor PHPCS configuration and rename test data providers'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/update-own-standard

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d5a6b58 and b91b870.

📒 Files selected for processing (4)

• phpcs.xml (1 hunks)
• tests/Unit/AbstractVariableSnakeCaseSniffTest.php (2 hunks)
• tests/Unit/LocalVariableSnakeCaseSniffTest.php (2 hunks)
• tests/Unit/ParameterSnakeCaseSniffTest.php (2 hunks)

🔇 Additional comments (4)

tests/Unit/LocalVariableSnakeCaseSniffTest.php (1)

47-47: LGTM! Consistent data provider renaming.

The data provider method renaming from providerProcessCases to dataProviderProcess improves naming consistency by adding the dataProvider prefix. The DataProvider attribute is correctly updated to match.

Also applies to: 66-66

tests/Unit/ParameterSnakeCaseSniffTest.php (1)

57-57: LGTM! Consistent data provider renaming.

The data provider method renaming from providerProcessCases to dataProviderProcess aligns with the naming convention improvements across the test suite. The DataProvider attribute correctly references the renamed method.

Also applies to: 76-76

tests/Unit/AbstractVariableSnakeCaseSniffTest.php (1)

30-30: LGTM! Improved data provider naming.

The data provider method renaming from providerSnakeCase to dataProviderSnakeCaseDetection enhances clarity by:

1. Adding the dataProvider prefix for consistency
2. Including "Detection" to better describe the test's purpose

The DataProvider attribute is correctly updated to match the new method name.

Also applies to: 46-46

phpcs.xml (1)

10-10: No changes needed—specific rule names remain valid after consolidation.

The consolidation is safe. The ruleset.xml explicitly defines both DrevOps.NamingConventions.LocalVariableSnakeCase and DrevOps.NamingConventions.ParameterSnakeCase rules. When phpcs.xml references <rule ref="DrevOps"/>, it loads these specific rules from the ruleset, making them still addressable by test configurations and PHPCS.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (d5a6b58) to head (b91b870).
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff            @@
##              main        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         6           
  Lines          387       387           
=========================================
  Hits           387       387           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.