Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Feature-gate test-programs#600

Merged
kubkon merged 1 commit intobytecodealliance:masterfrom
kubkon:test-program-feature-gated
Nov 21, 2019
Merged

Feature-gate test-programs#600
kubkon merged 1 commit intobytecodealliance:masterfrom
kubkon:test-program-feature-gated

Conversation

@kubkon
Copy link
Member

@kubkon kubkon commented Nov 19, 2019

This commit feature-gates test-programs as a workaround for
cargo test not allowing for optional dev-dependencies.

This commit addresses #595.

@kubkon kubkon added the wasi:tests Issues pertaining to WASI tests in Wasmtime label Nov 19, 2019
@kubkon kubkon force-pushed the test-program-feature-gated branch from 05ff80e to 8b68a14 Compare November 20, 2019 08:40
This commit feature-gates `test-programs` as a workaround for
`cargo test` not allowing for optional dev-dependencies.

This commit addresses bytecodealliance#595.
@kubkon kubkon force-pushed the test-program-feature-gated branch from 8b68a14 to b2fddf3 Compare November 20, 2019 08:53
@kubkon
Copy link
Member Author

kubkon commented Nov 21, 2019

I'm gonna go ahead and merge this PR, and close #595. If at a later date we decide we won't change things in a different direction, let's reopen #595 and take it from there!

@kubkon kubkon merged commit a58709d into bytecodealliance:master Nov 21, 2019
@kubkon kubkon deleted the test-program-feature-gated branch November 21, 2019 06:01
kubkon pushed a commit to kubkon/wasmtime that referenced this pull request Nov 21, 2019
This commit fixes bugs in enabling feature-gating of `test-programs`
which was introduced in bytecodealliance#600. It turns out, bytecodealliance#600 accidentally
disabled `test-programs` from ever running, and this commit fixes
that.
kubkon pushed a commit that referenced this pull request Nov 21, 2019
* Fix feature-gating of test-programs

This commit fixes bugs in enabling feature-gating of `test-programs`
which was introduced in #600. It turns out, #600 accidentally
disabled `test-programs` from ever running, and this commit fixes
that.

* Fix the CI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wasi:tests Issues pertaining to WASI tests in Wasmtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants