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

Add feature gate tests #24749

Merged
merged 4 commits into from
Apr 25, 2015
Merged

Add feature gate tests #24749

merged 4 commits into from
Apr 25, 2015

Conversation

lgrz
Copy link
Contributor

@lgrz lgrz commented Apr 23, 2015

As part of the audit for #22820 the following feature gate tests have been
added:

  • negate_unsigned
  • on_unimplemented
  • optin_builtin_traits
  • plugin
  • rustc_attrs
  • rustc_diagnostic_macros
  • slice_patterns

In addition some feature gate error message typos fixed.

@rust-highfive
Copy link
Collaborator

r? @nrc

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

@nrc
Copy link
Member

nrc commented Apr 23, 2015

It looks to me like a couple of tests got renamed when they should have been copied, the end result is we lose two tests. Is that accidental or deliberate?

@lgrz
Copy link
Contributor Author

lgrz commented Apr 23, 2015

Thanks, I was just about to comment on this. The diff on Github shows files as being renamed, however the individual commits themselves do no such thing. Not sure what happened, I'll look into this further.

@lgrz
Copy link
Contributor Author

lgrz commented Apr 23, 2015

There is a commit that deletes some duplicate tests (1ec6bb7) and these deleted files are showing up as being renamed in the diff on Github. But locally a git diff feature-gate-22820...master shows up fine (without the renaming).

I am going to split out the deletions and submit them as a separate PR as having the rename in the history on Github is not the intended meaning of the changeset. Hopefully that will make the changes appear as intended on Github.

@lgrz
Copy link
Contributor Author

lgrz commented Apr 24, 2015

Sorry there was an error in that latest push, fixing now...

As part of the audit for #22820 the following feature gate tests have
been added:

* `negate_unsigned`
* `on_unimplemented`
* `optin_builtin_traits`
* `plugin`
* `rustc_attrs`
* `slice_patterns`
As part of the audit for #22820 the following feature gate tests have
been added:

* `rustc_diagnostic_macros`

/cc #19624
@lgrz
Copy link
Contributor Author

lgrz commented Apr 24, 2015

Ok, sorry about mixup there, I've removed the duplicate tests that were deleted from this PR, I will submit them in a separate PR. Thanks.

@nrc
Copy link
Member

nrc commented Apr 24, 2015

@bors r+ 0cae315 rollup

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Apr 25, 2015
As part of the audit for rust-lang#22820 the following feature gate tests have been
added:

* `negate_unsigned`
* `on_unimplemented`
* `optin_builtin_traits`
* `plugin`
* `rustc_attrs`
* `rustc_diagnostic_macros`
* `slice_patterns`

In addition some feature gate error message typos fixed.
bors added a commit that referenced this pull request Apr 25, 2015
@bors bors merged commit 0cae315 into rust-lang:master Apr 25, 2015
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