Skip to content

[FEATURE] Introduce #[RequiresClass] attribute #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 24, 2024

Conversation

eliashaeussler
Copy link
Owner

This PR introduces a new attribute: #[RequiresClass]

It can be used to define a dependency to a given class. Tests will be skipped if this class cannot be loaded by the current autoloader.

@eliashaeussler eliashaeussler added the enhancement New feature or request label Aug 24, 2024
@eliashaeussler eliashaeussler self-assigned this Aug 24, 2024
@eliashaeussler eliashaeussler force-pushed the feature/requires-class-attribute branch from 66ff313 to b270351 Compare August 24, 2024 12:16
Copy link

qlty-cloud-legacy bot commented Aug 24, 2024

Code Climate has analyzed commit 039bd7f and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 100.0% (0.0% change).

View more on Code Climate.

@eliashaeussler eliashaeussler force-pushed the feature/requires-class-attribute branch from b270351 to 039bd7f Compare August 24, 2024 12:18
@eliashaeussler eliashaeussler merged commit 4acff6a into main Aug 24, 2024
21 checks passed
@eliashaeussler eliashaeussler deleted the feature/requires-class-attribute branch August 24, 2024 12:19
@coveralls
Copy link
Collaborator

coveralls commented Aug 24, 2024

Pull Request Test Coverage Report for Build 10538398885

Details

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

Totals Coverage Status
Change from base Build 10538040975: 0.0%
Covered Lines: 165
Relevant Lines: 165

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants