Skip to content

Conversation

jonas-schievink
Copy link
Contributor

#[link(cfg)] is perma-unstable and is not documented anywhere else. It makes more sense to mention wasm_import_module here since it's stable.

This makes it harder to hit #70538 (if it weren't for this text, I wouldn't even know this feature existed).

@rust-highfive
Copy link
Contributor

r? @varkor

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 29, 2020
@varkor
Copy link
Contributor

varkor commented Mar 29, 2020

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Mar 29, 2020

📌 Commit 042f494 has been approved by varkor

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 29, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 29, 2020
Rollup of 4 pull requests

Successful merges:

 - rust-lang#69702 (Rename TyLayout to TyAndLayout.)
 - rust-lang#70539 (add test for 62220)
 - rust-lang#70540 (#[link]: mention wasm_import_module instead of cfg)
 - rust-lang#70541 (prohibit_generics: update has_err for consts)

Failed merges:

r? @ghost
@bors bors merged commit 1bf4873 into rust-lang:master Mar 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants