Skip to content

Conversation

sanmai
Copy link
Owner

@sanmai sanmai commented Jul 28, 2025

Declares that DuoClock provides psr/clock-implementation to help package managers identify this as a PSR-20 compatible clock.

Declares that DuoClock provides psr/clock-implementation to help
package managers identify this as a PSR-20 compatible clock.
@sanmai sanmai requested a review from Copilot July 28, 2025 01:45
Copy link

@Copilot 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 declares PSR-20 clock implementation compatibility by adding a "provide" section to composer.json, indicating that the DuoClock package provides a PSR-20 compatible clock implementation.

  • Adds composer.json "provide" declaration for psr/clock-implementation
Comments suppressed due to low confidence (1)

composer.json:28

  • The PSR-20 specification uses semantic versioning starting with version 1.0.0, not 1.0. The provide declaration should specify "1.0.0" to match the actual PSR-20 specification version.
        "psr/clock-implementation": "1.0"

@coveralls
Copy link

coveralls commented Jul 28, 2025

Pull Request Test Coverage Report for Build 16558072450

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 16283314973: 0.0%
Covered Lines: 16
Relevant Lines: 16

💛 - Coveralls

2 similar comments
@coveralls
Copy link

Pull Request Test Coverage Report for Build 16558072450

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 16283314973: 0.0%
Covered Lines: 16
Relevant Lines: 16

💛 - Coveralls

@coveralls
Copy link

Pull Request Test Coverage Report for Build 16558072450

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 16283314973: 0.0%
Covered Lines: 16
Relevant Lines: 16

💛 - Coveralls

@sanmai sanmai merged commit 30aa400 into main Jul 28, 2025
6 of 7 checks passed
@sanmai sanmai deleted the fix/provides branch July 28, 2025 02:17
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