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

LGTM.com errors #2851

Merged
merged 3 commits into from Oct 10, 2021
Merged

LGTM.com errors #2851

merged 3 commits into from Oct 10, 2021

Commits on Oct 9, 2021

  1. LGTM.com error: Modification of parameter with default 

    Default value flows to here and is mutated.
    @DimitriPapadopoulos
    DimitriPapadopoulos committed Oct 9, 2021
    Copy the full SHA
    4b2c563 View commit details
    Browse the repository at this point in the history

  2. LGTM.com error: Wrong number of arguments in a class instantiation 

    Call to ReadError.__init__ with too few arguments; should be no fewer than 2.
    @DimitriPapadopoulos
    DimitriPapadopoulos committed Oct 9, 2021
    Copy the full SHA
    f554129 View commit details
    Browse the repository at this point in the history

Commits on Oct 10, 2021

  1. Add a unit test, fix typos

    @dannysepler
    dannysepler committed Oct 10, 2021
    Copy the full SHA
    8ce97b2 View commit details
    Browse the repository at this point in the history