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

Simpler dependencies #353

Merged
merged 4 commits into from
Dec 6, 2021
Merged

Simpler dependencies #353

merged 4 commits into from
Dec 6, 2021

Conversation

dvdplm
Copy link
Contributor

@dvdplm dvdplm commented Dec 3, 2021

Micro PR with outcome of some code reading and tinkering.

Moves sp-arithmetic to dev-dependencies and switches to use default-features = false. Was hoping for a reduced number of dependent crates but it stays the same.

Add sp-arithmetic to dev-dependencies
Don't use any default features
@dvdplm dvdplm self-assigned this Dec 3, 2021
@dvdplm dvdplm requested review from insipx and jsdw December 3, 2021 17:19
Copy link
Collaborator

@jsdw jsdw left a comment

Choose a reason for hiding this comment

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

LGTM! (I like Niklas's suggestions also)

dvdplm and others added 3 commits December 6, 2021 11:17
Co-authored-by: Niklas Adolfsson <niklasadolfsson1@gmail.com>
Co-authored-by: Niklas Adolfsson <niklasadolfsson1@gmail.com>
Co-authored-by: Niklas Adolfsson <niklasadolfsson1@gmail.com>
@dvdplm dvdplm merged commit 55aafa2 into master Dec 6, 2021
@dvdplm dvdplm deleted the dp-misc-deps branch December 6, 2021 10:17
0623forbidden pushed a commit to DEIPworld/substrate-subxt that referenced this pull request Feb 15, 2022
* Don't re-export sp-arithmetic
Add sp-arithmetic to dev-dependencies
Don't use any default features

* Update Cargo.toml

Co-authored-by: Niklas Adolfsson <niklasadolfsson1@gmail.com>

* Update Cargo.toml

Co-authored-by: Niklas Adolfsson <niklasadolfsson1@gmail.com>

* Update Cargo.toml

Co-authored-by: Niklas Adolfsson <niklasadolfsson1@gmail.com>

Co-authored-by: Niklas Adolfsson <niklasadolfsson1@gmail.com>
# 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.

4 participants