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

Fixes bug in wasm-builder with cargo publish #7299

Merged
merged 2 commits into from Oct 12, 2020

Conversation

@bkchr
Copy link
Contributor

@bkchr bkchr commented Oct 10, 2020

There was a bug in wasm-builder which resulted in generating a
Cargo.lock in the project directory because of running cargo metadata. This resulted in commands like cargo publish to fail (if
there was no Cargo.lock before building), because it checks that the
project directory isn't modified.

Fixes: #7116

There was a bug in wasm-builder which resulted in generating a
`Cargo.lock` in the project directory because of running `cargo
metadata`. This resulted in commands like `cargo publish` to fail (if
there was no `Cargo.lock` before building), because it checks that the
project directory isn't modified.
@andresilva
Copy link
Member

@andresilva andresilva commented Oct 12, 2020

New version needs to be published and should be good to merge afterwards.

Co-authored-by: André Silva <123550+andresilva@users.noreply.github.com>
@bkchr bkchr merged commit 8efe8b8 into master Oct 12, 2020
20 checks passed
20 checks passed
check_status
Details
continuous-integration/gitlab-build-rust-doc Build stage: build; status: success
Details
continuous-integration/gitlab-cargo-check-benches Build stage: test; status: success
Details
continuous-integration/gitlab-cargo-check-macos Build stage: test; status: success
Details
continuous-integration/gitlab-cargo-check-subkey Build stage: test; status: success
Details
continuous-integration/gitlab-check-labels Build stage: .post; status: success
Details
continuous-integration/gitlab-check-line-width Build stage: check; status: success
Details
continuous-integration/gitlab-check-polkadot-companion-build Build stage: build; status: success
Details
continuous-integration/gitlab-check-polkadot-companion-status Build stage: build; status: success
Details
continuous-integration/gitlab-check-runtime Build stage: check; status: success
Details
continuous-integration/gitlab-check-web-wasm Build stage: test; status: success
Details
continuous-integration/gitlab-skip-if-draft Build stage: .pre; status: success
Details
continuous-integration/gitlab-test-browser-node Build stage: build; status: success
Details
continuous-integration/gitlab-test-dependency-rules Build stage: check; status: success
Details
continuous-integration/gitlab-test-deterministic-wasm Build stage: test; status: success
Details
continuous-integration/gitlab-test-frame-examples-compile-to-wasm Build stage: test; status: success
Details
continuous-integration/gitlab-test-full-crypto-feature Build stage: test; status: success
Details
continuous-integration/gitlab-test-linux-stable Build stage: test; status: success
Details
continuous-integration/gitlab-test-linux-stable-int Build stage: test; status: success
Details
continuous-integration/gitlab-test-prometheus-alerting-rules Build stage: test; status: success
Details
@bkchr bkchr deleted the bkchr-wasm-builder-fix-cargo-publish branch Oct 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.