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

Specific case for ParametersIndentationViolation #1021

Closed
AlwxSin opened this issue Nov 25, 2019 · 2 comments · Fixed by #1022
Closed

Specific case for ParametersIndentationViolation #1021

AlwxSin opened this issue Nov 25, 2019 · 2 comments · Fixed by #1022
Labels
bug Something isn't working documentation Docs related task pr-merged

Comments

@AlwxSin
Copy link
Contributor

AlwxSin commented Nov 25, 2019

I have a lot of violations fro my pytest fixtures like this one

@pytest.mark.parametrize(
    ("is_verified", "status_code"),
    [
        (True, web.HTTPOk.status_code),
        (False, web.HTTPUnauthorized.status_code),
    ])

It's unclear from doc examples how to fix it. Documentation contains only simple examples.

I suggest adding one complex example like the above one.

@sobolevn sobolevn added bug Something isn't working documentation Docs related task labels Nov 25, 2019
@sobolevn sobolevn added this to the Version 0.13.x milestone Nov 25, 2019
@sobolevn
Copy link
Member

sobolevn commented Nov 25, 2019

@pytest.mark.parametrize(('is_verified', 'status_code'), [
     (True, web.HTTPOk.status_code),
     (False, web.HTTPUnauthorized.status_code),
])

Will you please update the docs?

@AlwxSin
Copy link
Contributor Author

AlwxSin commented Nov 25, 2019

Of course. #1022

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Docs related task pr-merged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants