Skip to content
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

move SparseArrays to the external repo https://github.com/JuliaLang/SparseArrays.jl #43813

Merged
merged 3 commits into from
Jan 14, 2022

Conversation

KristofferC
Copy link
Member

@KristofferC KristofferC commented Jan 14, 2022

Right now it uses the last commit in JuliaSparse/SparseArrays.jl#7. That should be merged and the SHA updated when it is.

There are some tests that loads SparseArrays:

  • Random/test/runtests.jl
  • abstractarray.jl
  • ambiguous.jl
  • arrayops.jl
  • core.jl
  • hashing.jl
  • show.jl

These tests should migrate to the SparseArray repo, but that can be done after this PR is done.

@dkarrasch, @ViralBShah, I think we should try not to merge anything to SparseArrays in the Julia repo until this PR is merged.

@KristofferC KristofferC requested review from staticfloat, ViralBShah and dkarrasch and removed request for staticfloat January 14, 2022 08:55
Copy link
Member

@dkarrasch dkarrasch left a comment

Choose a reason for hiding this comment

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

I'm not an expert in the package management, but tests pass, so this LGTM. Do we need this PR bot which files a checksum-update PR every time we merge a PR to SparseArrays.jl?

@dkarrasch dkarrasch added sparse Sparse arrays stdlib Julia's standard library labels Jan 14, 2022
@KristofferC
Copy link
Member Author

KristofferC commented Jan 14, 2022

Merged JuliaSparse/SparseArrays.jl#7 and updated the SHA here. Will try merge somewhat quickly so that conflicts doesn't occur.

@ViralBShah
Copy link
Member

Do we want a NEWS entry?

@KristofferC
Copy link
Member Author

Sounds like a good idea, I'll add one.

@KristofferC KristofferC merged commit b57d2e1 into master Jan 14, 2022
@KristofferC KristofferC deleted the kc/sparse_separate_repo branch January 14, 2022 15:38
@staticfloat
Copy link
Member

Wish I could give this PR a posthumous approval!

@DilumAluthge
Copy link
Member

DilumAluthge commented Jan 14, 2022

Do we need this PR bot which files a checksum-update PR every time we merge a PR to SparseArrays.jl?

Are you referring to https://github.com/JuliaLang/BumpStdlibs.jl?

BumpStdlibs.jl won't bump the checksums every time you merge a PR to SparseArrays.jl. Instead, when you want to trigger a bump manually, just follow the instructions in the BumpStdlibs README.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sparse Sparse arrays stdlib Julia's standard library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants