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

Fix the ffi_unwind_calls lint documentation #108588

Merged
merged 3 commits into from Mar 23, 2023
Merged

Fix the ffi_unwind_calls lint documentation #108588

merged 3 commits into from Mar 23, 2023

Commits on Mar 1, 2023

  1. Add check for errant {{produces}} marker. 

    If a lint example has an `ignore` tag, but the lint author also includes
the {{produces}} marker, then the output will just contain the text
`{{produces}}`. This adds a check for this mistake and provides help on
how the lint docs should be written.
    @ehuss
    ehuss committed Mar 1, 2023
    Configuration menu
    Copy the full SHA
    c6cba68 View commit details
    Browse the repository at this point in the history

  2. Fix the ffi_unwind_calls lint documentation

    @ehuss
    ehuss committed Mar 1, 2023
    Configuration menu
    Copy the full SHA
    15450b1 View commit details
    Browse the repository at this point in the history

  3. Collect all matching messages for a lint. 

    Some examples may contain multiple lines which trigger the lint.
Previously it would only display the first message.
This updates it so that all matching instances of the lint are displayed.
    @ehuss
    ehuss committed Mar 1, 2023
    Configuration menu
    Copy the full SHA
    ab2508a View commit details
    Browse the repository at this point in the history