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

Plugins deprecation: don’t suggest simply removing the attribute #65498

Merged
merged 1 commit into from
Oct 19, 2019

Commits on Oct 17, 2019

  1. Plugins deprecation: don’t suggest simply removing the attribute

    Building Servo with a recent Nightly produces:
    
    ```rust
    warning: use of deprecated attribute `plugin`: compiler plugins are deprecated. See rust-lang#29597
      --> components/script/lib.rs:14:1
       |
    14 | #![plugin(script_plugins)]
       | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove this attribute
       |
       = note: `#[warn(deprecated)]` on by default
    ```
    
    First, linking to rust-lang#29597 is not ideal
    since there is pretty much no discussion there of the deprecation
    and what can be used instead.
    This PR changes the link to the deprecation PR which does have more discussion.
    
    Second, the “remove this attribute” suggestion is rather unhelpful.
    Just because a feature is deprecated doesn’t mean that simply removing its use
    without a replacement is acceptable.
    
    In the case of custom lint, there is no replacement available.
    Prefixing a message with “help:” when telling users that they’re screwed
    honestly feels disrespectful.
    
    This PR also changes the message to be more factual.
    SimonSapin committed Oct 17, 2019
    Configuration menu
    Copy the full SHA
    71b0049 View commit details
    Browse the repository at this point in the history