Skip to content
New issue

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

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

Already on GitHub? # to your account

Try to sort out the homs #1488

Merged
merged 3 commits into from
May 3, 2024
Merged

Try to sort out the homs #1488

merged 3 commits into from
May 3, 2024

Conversation

lgoettgens
Copy link
Contributor

cf. oscar-system/Oscar.jl#3667 (comment)

Alternative to and thus closes #1483.

Copy link

codecov bot commented May 3, 2024

Codecov Report

Attention: Patch coverage is 18.18182% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 75.31%. Comparing base (c19f1c5) to head (43078a9).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1488      +/-   ##
==========================================
+ Coverage   75.26%   75.31%   +0.05%     
==========================================
  Files         354      354              
  Lines      112724   112732       +8     
==========================================
+ Hits        84845    84908      +63     
+ Misses      27879    27824      -55     
Files Coverage Δ
src/Deprecations.jl 100.00% <ø> (ø)
src/GrpAb/GrpAbFinGen.jl 76.03% <18.18%> (-0.54%) ⬇️

... and 29 files with indirect coverage changes

@lgoettgens lgoettgens marked this pull request as ready for review May 3, 2024 17:27
@thofma
Copy link
Owner

thofma commented May 3, 2024

why is Aqua unhappy?

@lgoettgens
Copy link
Contributor Author

why is Aqua unhappy?

The persistent task test does not work with deved dependecies

@thofma
Copy link
Owner

thofma commented May 3, 2024

so just ignore it?

@lgoettgens
Copy link
Contributor Author

Jup, should be fine. And even if not, we are not changing anything for oscar master here

@thofma thofma merged commit c9da9fa into thofma:master May 3, 2024
30 of 34 checks passed
@lgoettgens lgoettgens deleted the lg/compatibility branch May 3, 2024 21:09
@lgoettgens lgoettgens mentioned this pull request May 4, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants