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: pin orml to specific revision #39

Merged
merged 2 commits into from May 28, 2021
Merged

chore: pin orml to specific revision #39

merged 2 commits into from May 28, 2021

Conversation

enthusiastmartin
Copy link
Contributor

Description

Basilisk uses orml dependencies from master branch. Since there was a change to use more recent substrate version from different branch ( polkadot-v0.9.2) - doing any cargo update will result in having 2 different versions of substrate.

Basilisk is currently built with substrate from branch polkadot-v0.9.1

Therefore, this PR pins the orml to specific revision where the same version of substrate is used.

@mrq1911
Copy link
Member

mrq1911 commented May 26, 2021

@martinfridrich why we just dont upgrade to 0.9.2? well have to upgrade to 0.9.3 anyway soon

@enthusiastmartin
Copy link
Contributor Author

@martinfridrich why we just dont upgrade to 0.9.2? well have to upgrade to 0.9.3 anyway soon

yes, sure. we can update. but as always,i'd expect some effort and this was jus to make sure that basilisk works ok even when it needs some updates.

i would say that it would be probably good to have orml pinned some specific revision and update when necessary.

@mrq1911 mrq1911 merged commit 9367f11 into master May 28, 2021
@mrq1911 mrq1911 deleted the chore/orml_rev_pin branch October 4, 2021 19:38
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.

None yet

2 participants