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.randvars #667

Closed
2 tasks
pnkraemer opened this issue Mar 23, 2022 · 1 comment · Fixed by #684
Closed
2 tasks

Easy-to-fix pylint messages in probnum.randvars #667

pnkraemer opened this issue Mar 23, 2022 · 1 comment · Fixed by #684
Labels
good first issue Good for newcomers randvars Issues related to random variables

Comments

@pnkraemer
Copy link
Collaborator

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.randvars subpackage, these are

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

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.
@pnkraemer pnkraemer added good first issue Good for newcomers randvars Issues related to random variables labels Mar 23, 2022
@NDOWAH
Copy link

NDOWAH commented Mar 28, 2022

I would like to work on this issue.

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 randvars Issues related to random variables
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants