Skip to content
This repository was archived by the owner on May 15, 2025. It is now read-only.

Clean up the testing #144

Merged
merged 1 commit into from
May 17, 2024
Merged

Clean up the testing #144

merged 1 commit into from
May 17, 2024

Conversation

avik-pal
Copy link
Member

No description provided.

test/runtests.jl Outdated
if GROUP == "GPU"
ReTestItems.runtests(joinpath(@__DIR__, "gpu/"))
if GROUP == "All"
ReTestItems.runtests(@__DIR__; tags=[:all])
Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
ReTestItems.runtests(@__DIR__; tags=[:all])
ReTestItems.runtests(@__DIR__; tags = [:all])

Copy link
Member Author

Choose a reason for hiding this comment

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

@ChrisRackauckas we should enable this new JuliaFormatter action across all the repos, the feedback is so much better than just a failure

Copy link
Member

Choose a reason for hiding this comment

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

Yes, I've already talked to @thazhemadam about rolling it out to all repos. I'll be happy when it's there, definitely a good idea.

@avik-pal avik-pal force-pushed the ap/testing branch 5 times, most recently from 9b13c28 to e30ae67 Compare May 17, 2024 15:52
Copy link

codecov bot commented May 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.58%. Comparing base (34e7d8c) to head (7e999d6).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #144   +/-   ##
=======================================
  Coverage   90.58%   90.58%           
=======================================
  Files          23       23           
  Lines        1296     1296           
=======================================
  Hits         1174     1174           
  Misses        122      122           

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

@avik-pal avik-pal force-pushed the ap/testing branch 2 times, most recently from 7bf9501 to 09a7691 Compare May 17, 2024 18:17
@avik-pal avik-pal merged commit c3d64b3 into main May 17, 2024
18 of 19 checks passed
@avik-pal avik-pal deleted the ap/testing branch May 17, 2024 20:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants