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.

Sign up for GitHub

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

Jump to bottom

Lint on incorrect unseparated repetition #55373

Closed
wants to merge 7 commits into from
Closed

Lint on incorrect unseparated repetition #55373

wants to merge 7 commits into from

Commits on Dec 6, 2018

  1. Disallow incorrect unseparated repetition 

    Macro pattern `($a:expr, $b:expr)` is invalid, for good reason.
Correctly reject pattern `($($a:expr)*)`, as it is functionally the
same.
    @estebank
    estebank committed Dec 6, 2018
    Configuration menu
    Copy the full SHA
    d55936d View commit details
    Browse the repository at this point in the history

  2. Add suggestion

    @estebank
    estebank committed Dec 6, 2018
    Configuration menu
    Copy the full SHA
    2c1008c View commit details
    Browse the repository at this point in the history

  3. Turn error into a warning

    @estebank
    estebank committed Dec 6, 2018
    Configuration menu
    Copy the full SHA
    0cc8969 View commit details
    Browse the repository at this point in the history

  4. Tweak unexpected macro arg tt panic message

    @estebank
    estebank committed Dec 6, 2018
    Configuration menu
    Copy the full SHA
    ae79ff0 View commit details
    Browse the repository at this point in the history

  5. Formatting changes

    @estebank
    estebank committed Dec 6, 2018
    Configuration menu
    Copy the full SHA
    97bda67 View commit details
    Browse the repository at this point in the history

  6. Turn warning into warn-by-default lint

    @estebank
    estebank committed Dec 6, 2018
    Configuration menu
    Copy the full SHA
    ca5e89d View commit details
    Browse the repository at this point in the history

Commits on Dec 7, 2018

  1. Turn lint to allow by default and add future proofing warning

    @estebank
    estebank committed Dec 7, 2018
    Configuration menu
    Copy the full SHA
    fa976ba View commit details
    Browse the repository at this point in the history