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

Easy-to-fix pylint messages in probnum.linalg #671

Open
1 of 4 tasks
pnkraemer opened this issue Mar 23, 2022 · 0 comments
Open
1 of 4 tasks

Easy-to-fix pylint messages in probnum.linalg #671

pnkraemer opened this issue Mar 23, 2022 · 0 comments
Labels
good first issue Good for newcomers linalg Issues related to linear algebra

Comments

@pnkraemer
Copy link
Collaborator

pnkraemer commented Mar 23, 2022

Context:
There is ongoing work on refactoring probnum to comply with the desired pylint configuration. See #523 for context.
Many of those refactorings should be very simple to resolve.

Details:
For the probnum.linalg subpackage, these are

  • line-too-long (some docstrings have too long lines)
  • no-else-return
  • no-else-raise
  • else-if-used

Once a message has been resolved, please remove the explicit pylint-ignore on the bottom of the tox.ini file.

Some notes, if you would like to take this on:

  • Have a look at the contribution guide before opening a pull request.
  • Please strongly consider resolving each message in a separate pull request. This makes the reviews go more smoothly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers linalg Issues related to linear algebra
Projects
None yet
Development

No branches or pull requests

1 participant