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

Install patched version of sandpaper #2

Merged
merged 1 commit into from
May 28, 2024
Merged

Conversation

Bisaloo
Copy link
Contributor

@Bisaloo Bisaloo commented May 28, 2024

Note that I don't advocate using forks from the official sandpaper in the long run. This is just a short-term patch which will hopefully gather support to merge carpentries/sandpaper#533 with confidence.

Dependency management in sandpaper is very powerful but also very complex and I think carpentries/sandpaper#533 hasn't been merged out of caution of potential side effects it may have on other portions of the code.

In my organization, we have used this patch for more than 6 months without issues now so it's probably worth a try.

Copy link

🆗 Pre-flight checks passed 😃

This pull request has been checked and contains no modified workflow files or spoofing.

It should be safe to Approve and Run the workflows that need maintainer approval.

@dmgatti
Copy link
Collaborator

dmgatti commented May 28, 2024

Merging this in to see whether it will fix our build issues with spacexr and sparseMatrixStats.

@dmgatti dmgatti merged commit a400eb6 into smcclatchy:main May 28, 2024
2 of 3 checks passed
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