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 #39250

Merged
merged 1 commit into from
Jan 31, 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 23, 2017

Thanks!

As I said in #39059 we need to wait until #39247 is merged (+ then you'll have to rebase), as it requires searching inside compile-fail-fulldeps to be implemented.

@cseale
Copy link
Contributor Author

cseale commented Jan 23, 2017

np

@est31
Copy link
Member

est31 commented Jan 23, 2017

@cseale the PR got merged.

@bors
Copy link
Contributor

bors commented Jan 26, 2017

☔ The latest upstream changes (presumably #39075) made this pull request unmergeable. Please resolve the merge conflicts.

@@ -166,8 +166,7 @@ pub fn check(path: &Path, bad: &mut bool) {

// FIXME get this whitelist empty.
let whitelist = vec![
"abi_ptx", "simd", "macro_reexport",
"static_recursion", "quote",
"abi_ptx", "simd", "static_recursion", "reflect",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was "reflect" added here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was removed by #39075, so shouldn't be present.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cleaned this up

@@ -166,8 +166,7 @@ pub fn check(path: &Path, bad: &mut bool) {

// FIXME get this whitelist empty.
let whitelist = vec![
"abi_ptx", "simd", "macro_reexport",
"static_recursion", "quote",
"abi_ptx", "simd", "static_recursion",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You've got a trailing whitespace here, that's why tidy fails.

@jseyfried
Copy link
Contributor

@bors delegate=est31

@bors
Copy link
Contributor

bors commented Jan 29, 2017

✌️ @est31 can now approve this pull request

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

r? @est31
@est31
Copy link
Member

est31 commented Jan 31, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Jan 31, 2017

📌 Commit 94e9086 has been approved by est31

@bors
Copy link
Contributor

bors commented Jan 31, 2017

⌛ Testing commit 94e9086 with merge b854d74...

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

Removal of the lang feature gate tests whitelist #39059

r? @est31
@bors
Copy link
Contributor

bors commented Jan 31, 2017

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

@bors bors merged commit 94e9086 into rust-lang:master Jan 31, 2017
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