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

Use Declarations in More Places #1976

Closed
wants to merge 4 commits into from

Commits on Apr 18, 2017

  1. Configuration menu
    Copy the full SHA
    ef4e911 View commit details
    Browse the repository at this point in the history

Commits on May 2, 2017

  1. Remove use in match, add use in trait; Add grammar

    Remove `use` in `match`. Should be as own RFC, and not willing to defend it personally.
    Likewise, removed parts in the how we teach and drawbacks, making those sections less
    "if"fy.
    
    Added `trait`, as that's analogous to `impl` and I feel stupid for not doing that initially.
    
    Added grammar changes. Not sure if they're the right changes, but at least they exist.
    Havvy committed May 2, 2017
    Configuration menu
    Copy the full SHA
    338b9c0 View commit details
    Browse the repository at this point in the history

Commits on May 16, 2017

  1. Disallow pub use in these new areas.

    There's RFCs for having `pub use` in traits work as aliases, and
    allowing `pub use` would be a back-compat hazard for these RFCs.
    
    Sorry trait authors.
    Havvy committed May 16, 2017
    Configuration menu
    Copy the full SHA
    32d0c3f View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    eecab3d View commit details
    Browse the repository at this point in the history