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.randprocs #668

Closed
1 task
pnkraemer opened this issue Mar 23, 2022 · 0 comments · Fixed by #681
Closed
1 task

Easy-to-fix pylint messages in probnum.randprocs #668

pnkraemer opened this issue Mar 23, 2022 · 0 comments · Fixed by #681
Labels
good first issue Good for newcomers randprocs Issues related to random processes

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

  • line-too-long (some docstrings have too long lines)

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 randprocs Issues related to random processes labels Mar 23, 2022
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 randprocs Issues related to random processes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant