Skip to content

First try for coverage#47

Open
Arpit-Babbar wants to merge 7 commits intomasterfrom
add_coverage
Open

First try for coverage#47
Arpit-Babbar wants to merge 7 commits intomasterfrom
add_coverage

Conversation

@Arpit-Babbar
Copy link
Copy Markdown
Owner

No description provided.

@Arpit-Babbar Arpit-Babbar marked this pull request as ready for review December 8, 2025 19:37
@Arpit-Babbar Arpit-Babbar requested a review from Copilot December 8, 2025 19:37
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 adds code coverage reporting to the CI workflow using both Codecov and Coveralls. Coverage is collected only on Ubuntu builds to avoid redundant data collection across different operating systems.

  • Split test execution into coverage-enabled (Ubuntu) and coverage-disabled (macOS, Windows) paths
  • Process coverage data using julia-processcoverage action
  • Upload coverage reports to both Codecov and Coveralls services

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

Comment thread .github/workflows/CI.yml Outdated
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.


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

Comment thread .github/workflows/CI.yml
Comment thread .github/workflows/CI.yml
Comment thread .github/workflows/CI.yml
Comment thread .github/workflows/CI.yml
# At first, we check out the repository and download all artifacts
# (and list files for debugging).
- uses: actions/checkout@v6
- uses: actions/download-artifact@v6
Copy link

Copilot AI Dec 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a version mismatch between artifact actions. This workflow uses actions/upload-artifact@v5 (lines 106, 153) but actions/download-artifact@v6 here. For proper compatibility, both upload and download artifact actions should use the same major version (either both v5 or both v6). This mismatch may cause artifacts to not be found or downloaded correctly.

Suggested change
- uses: actions/download-artifact@v6
- uses: actions/download-artifact@v5

Copilot uses AI. Check for mistakes.
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