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

chore: do not bundle scip on docs.rs #81

Merged
merged 2 commits into from
Jan 8, 2025

Conversation

KnorpelSenf
Copy link
Contributor

Changes the build on docs.rs to avoid network access. This was introduced in #72 because bundling scip simplifies the build setup in both CI and locally.

Fixes #77.

@KnorpelSenf
Copy link
Contributor Author

KnorpelSenf commented Jan 7, 2025

@lovasoa one idea about this: we could introduce a new feature called all_default_offline_solvers which can be used on docs.rs. This might make it slightly more transparent how the two setups differ.

I did not perform this change because I don't like adding features just for ease of maintenance. But if you think it's better to have this, I can update the branch.

@lovasoa lovasoa merged commit 5048ac8 into rust-or:main Jan 8, 2025
1 check passed
@KnorpelSenf KnorpelSenf deleted the no-bundled-scip-on-docs.rs branch January 8, 2025 21:09
KnorpelSenf added a commit to KnorpelSenf/good_lp that referenced this pull request Jan 22, 2025
* chore: do not bundle scip on docs.rs

* docs: add comment on docs.rs features
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.

docs.rs build fails
2 participants