Skip to content

Conversation

@badasahog
Copy link
Contributor

replaces #7001

@codecov
Copy link

codecov bot commented May 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.69%. Comparing base (73d79ed) to head (73f08da).
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7011   +/-   ##
=======================================
  Coverage   98.69%   98.69%           
=======================================
  Files          79       79           
  Lines       14677    14678    +1     
=======================================
+ Hits        14486    14487    +1     
  Misses        191      191           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link

github-actions bot commented May 22, 2025

Comparison Plot

Generated via commit 73f08da

Download link for the artifact containing the test results: ↓ atime-results.zip

Task Duration
R setup and installing dependencies 4 minutes and 26 seconds
Installing different package versions 8 minutes and 22 seconds
Running and plotting the test cases 1 minutes and 58 seconds

@MichaelChirico
Copy link
Member

To clarify, this is just #7001, but pushed to this repo instead of your fork, right? It would help for historical reference to add that as a quick note to the PR description -- imagine there's some issue that git blame traces back to this PR in, say, 5 years. A quick added note of context in the PR would help a lot for such cases.

@MichaelChirico MichaelChirico merged commit ade5f95 into master May 26, 2025
11 checks passed
@MichaelChirico MichaelChirico deleted the constcastRename branch May 26, 2025 18:47
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