Skip to content

Test against downstream packages #260

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 25, 2023
Merged

Conversation

jishnub
Copy link
Member

@jishnub jishnub commented May 23, 2023

No description provided.

@codecov
Copy link

codecov bot commented May 23, 2023

Codecov Report

Merging #260 (31949c6) into master (d43ca84) will decrease coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #260      +/-   ##
==========================================
- Coverage   91.91%   91.78%   -0.13%     
==========================================
  Files          16       16              
  Lines        1459     1461       +2     
==========================================
  Hits         1341     1341              
- Misses        118      120       +2     

see 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jishnub
Copy link
Member Author

jishnub commented May 23, 2023

Test failure in InifniteLinearAlgebra seems to be due to c5b73d6, as the failing test is

  Expression: last(axes(b, 1))  RealInfinity()
   Evaluated: ℵ₀  +

Perhaps it's an issue with the test, and it should check for an integer instead?

@dlfivefifty
Copy link
Member

Yes the test should change. This change was needed to support rowsupport in another change you made in BandedMatrices.jl

@jishnub
Copy link
Member Author

jishnub commented May 24, 2023

Would it be ok to make me a member of this repo? I would like to get these tests working to catch downstream breakages earlier

@jishnub jishnub merged commit bc7074a into JuliaArrays:master May 25, 2023
@jishnub jishnub deleted the downstream branch May 25, 2023 16:42
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