Skip to content

Conversation

@oliverklee
Copy link
Collaborator

No description provided.

@coveralls
Copy link

Coverage Status

coverage: 62.467%. remained the same
when pulling b3cfe24 on task/followup
into 8be4011 on main.

@oliverklee oliverklee marked this pull request as draft December 2, 2025 12:22
@oliverklee oliverklee requested review from Copilot and removed request for JakeQZ December 2, 2025 12:22
Copilot finished reviewing on behalf of oliverklee December 2, 2025 12:24
Copy link

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 enhances the project's documentation and test infrastructure by adding PHPUnit code coverage annotations and polishing the changelog formatting for improved consistency.

  • Added @covers annotation to AtRuleTest for proper code coverage tracking
  • Improved changelog consistency by standardizing the wording pattern with "Add support for"
  • Enhanced changelog formatting by properly styling operators with backticks

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
tests/Unit/Property/AtRuleTest.php Added @covers annotation for code coverage metadata, consistent with other test files
CHANGELOG.md Polished changelog entries for consistency: standardized "Add support for" wording and added backticks around + and - operators for proper Markdown formatting

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

@JakeQZ
Copy link
Collaborator

JakeQZ commented Dec 2, 2025

  • Added @covers annotation to AtRuleTest for proper code coverage tracking

This was originally put in with the PR, but then removed because of the PHPStan warning.

I don't think @covers makes sense for an interface, because it has no code that can be covered.

@oliverklee oliverklee closed this Dec 4, 2025
@oliverklee oliverklee deleted the task/followup branch December 4, 2025 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants