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

fix(scripts) Fix publish script regarding wasmer-c-api #2076

Merged
merged 2 commits into from
Feb 4, 2021

Conversation

Hywan
Copy link
Contributor

@Hywan Hywan commented Jan 29, 2021

Description

Depends on #2053 (merged).

This patch is just 2 commits:

  • 52daf9b adding the WASMER_PUBLISH_SCRIPT_IS_RUNNING env var when running cargo publish,
  • 4f8bc8a to not run build.rs when the scripts/publish.py script is running.

Review

  • Add a short description of the the change to the CHANGELOG.md file not necessary

@Hywan Hywan added bug Something isn't working 📦 lib-c-api About wasmer-c-api 🤖 bot Bip bip labels Jan 29, 2021
@Hywan Hywan self-assigned this Jan 29, 2021
@Hywan
Copy link
Contributor Author

Hywan commented Feb 2, 2021

bors try

1 similar comment
@Hywan
Copy link
Contributor Author

Hywan commented Feb 4, 2021

bors try

bors bot added a commit that referenced this pull request Feb 4, 2021
@bors
Copy link
Contributor

bors bot commented Feb 4, 2021

@syrusakbary syrusakbary merged commit 42ee164 into wasmerio:master Feb 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 bot Bip bip bug Something isn't working 📦 lib-c-api About wasmer-c-api
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants