search

rustwasm / wee_alloc

The Wasm-Enabled, Elfin Allocator
Mozilla Public License 2.0
665 stars 49 forks source link

Fix feature flags for stable Rust testing from #65 #68

Closed RReverser closed 5 years ago

RReverser commented 5 years ago

size_classes is enabled by default, so one of the lines had to remain cargo check --release but another is supposed to be cargo check --release --no-default-features to test without size_classes enabled.

RReverser commented 5 years ago

Hmm, looks like there is a bunch of other places in build.sh, check.sh, appveyor.yml that use --features size_classes.

@fitzgen Does this mean they are all similarly broken (testing same configuration as default) or am I missing some switch?

fitzgen commented 5 years ago

@fitzgen Does this mean they are all similarly broken (testing same configuration as default) or am I missing some switch?

I think they are all misbehaving. Want to send a follow up PR?