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

[Gate Tests] - marking feature tests #39243

Merged
merged 1 commit into from
Jan 22, 2017
Merged

Conversation

cseale
Copy link
Contributor

@cseale cseale commented Jan 22, 2017

Removal of the lang feature gate tests whitelist #39059

r? @est31

@est31
Copy link
Member

est31 commented Jan 22, 2017

@cseale you should remove those tests from the whitelist:

"abi_ptx", "simd", "safe_suggestion", "macro_reexport",
"more_struct_aliases", "static_recursion", "reflect",
"quote", "cfg_target_has_atomic", "custom_attribute",
"default_type_parameter_fallback", "pushpop_unsafe",
"use_extern_macros", "staged_api", "const_indexing",
"unboxed_closures", "stmt_expr_attributes",
"cfg_target_thread_local", "unwind_attributes",
"inclusive_range_syntax"

Removal of the lang feature gate tests whitelist rust-lang#39059

r? @est31
@cseale
Copy link
Contributor Author

cseale commented Jan 22, 2017

@est31 done

@eddyb
Copy link
Member

eddyb commented Jan 22, 2017

@bors r=est31

@bors
Copy link
Contributor

bors commented Jan 22, 2017

📌 Commit ddb6027 has been approved by est31

@bors
Copy link
Contributor

bors commented Jan 22, 2017

⌛ Testing commit ddb6027 with merge a8fa310...

bors added a commit that referenced this pull request Jan 22, 2017
[Gate Tests] - marking feature tests

Removal of the lang feature gate tests whitelist #39059

r? @est31
@bors
Copy link
Contributor

bors commented Jan 22, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: est31
Pushing a8fa310 to master...

@bors bors merged commit ddb6027 into rust-lang:master Jan 22, 2017
@bors bors mentioned this pull request Jan 22, 2017
@est31
Copy link
Member

est31 commented Jan 22, 2017

Its merged. Thanks @cseale !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants