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

[fastapi] Implement fast-api-unused-path-parameter (FAST003) #12638

Merged
merged 24 commits into from
Aug 16, 2024
Merged

[fastapi] Implement fast-api-unused-path-parameter (FAST003) #12638

merged 24 commits into from
Aug 16, 2024

Commits on Aug 2, 2024

  1. Basic implementation of fastapi_unused_path_parameter

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    423206c View commit details
    Browse the repository at this point in the history

  2. Ignore path parameters that aren't valid identifiers

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    5168d72 View commit details
    Browse the repository at this point in the history

  3. Add some tests related to path convertors

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    b322603 View commit details
    Browse the repository at this point in the history

  4. Add tests with multiple path parameters

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    0c4ee27 View commit details
    Browse the repository at this point in the history

  5. Show the specific path causing the issue

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    18ca13f View commit details
    Browse the repository at this point in the history

  6. Add the add_parameter edit function

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    35ded68 View commit details
    Browse the repository at this point in the history

  7. Add automatic fixing, and improve message when the argument is presen… 

    …t as a pos-only arg
    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    4985888 View commit details
    Browse the repository at this point in the history

  8. Mark fast-api-unused-path-parameter as sometimes fixable

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    bc6346e View commit details
    Browse the repository at this point in the history

  9. Add fast-api-unused-path-parameter to the rule schema

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    7d4ac25 View commit details
    Browse the repository at this point in the history

  10. Add snapshot for fast-api-unused-path-parameter

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    ba47aaf View commit details
    Browse the repository at this point in the history

  11. Fix clippy errors

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    be57618 View commit details
    Browse the repository at this point in the history

  12. Fix docs formatting

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    5663e86 View commit details
    Browse the repository at this point in the history

Commits on Aug 3, 2024

  1. Use str instead of String where possible

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 3, 2024
    Configuration menu
    Copy the full SHA
    10e7381 View commit details
    Browse the repository at this point in the history

  2. Use iterators for extract_path_params_from_route, and move away from … 

    …regex
    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 3, 2024
    Configuration menu
    Copy the full SHA
    4a13fb4 View commit details
    Browse the repository at this point in the history

  3. Add an extra OK test

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 3, 2024
    Configuration menu
    Copy the full SHA
    4bb9fa8 View commit details
    Browse the repository at this point in the history

  4. Add some extra tests

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 3, 2024
    Configuration menu
    Copy the full SHA
    4b95723 View commit details
    Browse the repository at this point in the history

  5. Use the f-string parser

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 3, 2024
    Configuration menu
    Copy the full SHA
    a8e6aff View commit details
    Browse the repository at this point in the history

  6. Remove unused import

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 3, 2024
    Configuration menu
    Copy the full SHA
    739821e View commit details
    Browse the repository at this point in the history

Commits on Aug 4, 2024

  1. Ignore path parameters with conversions or debug text

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    77e0067 View commit details
    Browse the repository at this point in the history

  2. don't collect path_params into a vec

    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    f17e8df View commit details
    Browse the repository at this point in the history

Commits on Aug 13, 2024

  1. Few smaller nits

    @MichaReiser
    MichaReiser committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    8169cc4 View commit details
    Browse the repository at this point in the history

  2. Fix doc formatting

    @MichaReiser
    MichaReiser committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    79454db View commit details
    Browse the repository at this point in the history

  3. Revert to not using the f-string parser 

    This reverts commit a8e6aff, 77e0067, f17e8df.
    @Matthieu-LAURENT39
    Matthieu-LAURENT39 committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    9ec7632 View commit details
    Browse the repository at this point in the history

Commits on Aug 16, 2024

  1. Minor tweaks

    @charliermarsh
    charliermarsh committed Aug 16, 2024
    Configuration menu
    Copy the full SHA
    288bd10 View commit details
    Browse the repository at this point in the history