Skip to content

Upload coverage report#243

Merged
harikt merged 2 commits intoauraphp:6.xfrom
koriym:coverage-report
Apr 19, 2025
Merged

Upload coverage report#243
harikt merged 2 commits intoauraphp:6.xfrom
koriym:coverage-report

Conversation

@koriym
Copy link
Copy Markdown
Member

@koriym koriym commented Apr 19, 2025

No description provided.

koriym added 2 commits April 19, 2025 21:01
@koriym
Update CI workflow to include test coverage reporting
281ff92 
Added code coverage report generation using PHPUnit and integrated it with Codecov. This ensures better visibility and tracking of test coverage in the CI pipeline.
@koriym
Set newline at end of codecov.yml
6563840 
Added a missing newline at the end of the file to adhere to coding standards and improve compatibility. This ensures consistent behavior across different systems and tools.
@koriym koriym requested review from Copilot and harikt April 19, 2025 12:12
Copilot AI reviewed Apr 19, 2025
View reviewed changes
Copy link
Copy Markdown

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 updates the CI workflow to generate and upload a code coverage report.

  • The PHPUnit command now includes the "--coverage-clover=coverage.xml" flag to produce a coverage report.
  • The Codecov action is configured to upload the "coverage.xml" file.

@harikt harikt merged commit e48816f into auraphp:6.x Apr 19, 2025
1 of 2 checks passed
@harikt
Copy link
Copy Markdown
Member

harikt commented Apr 19, 2025

Thank you @koriym . Very helpful.

@harikt
Copy link
Copy Markdown
Member

harikt commented Apr 19, 2025 •
edited
Loading

We need to update the README.md to have the badge. Currently it seems not showing the coverage when I tried via url

@koriym koriym deleted the coverage-report branch April 19, 2025 14:09
@koriym
Copy link
Copy Markdown
Member Author

koriym commented Apr 19, 2025

[2025-04-19T14:08:58.718Z] ['error'] There was an error running the uploader: Error uploading to https://codecov.io: Error: There was an error fetching the storage URL during POST: 429 - {"message":"Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 2601s."}

https://github.com/auraphp/Aura.Sql/actions/runs/14549881194/job/40819810083

@harikt
Copy link
Copy Markdown
Member

harikt commented Apr 21, 2025

So this need a token in https://github.com/auraphp/Aura.Sql/blob/6.x/codecov.yml , right ?

@koriym
Copy link
Copy Markdown
Member Author

koriym commented Apr 21, 2025

Not with this file. See https://docs.codecov.com/docs/adding-the-codecov-token

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@harikt harikt Awaiting requested review from harikt

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@koriym @harikt