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

add msg to docs #31

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

add msg to docs #31

wants to merge 3 commits into from

Conversation

Borda
Copy link
Owner

@Borda Borda commented May 16, 2023

Before submitting
  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

Part of #1

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@Borda Borda added documentation Improvements or additions to documentation enhancement New feature or request labels May 16, 2023
@sonarcloud
Copy link

sonarcloud bot commented May 16, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@pep8speaks
Copy link

Hello @Borda! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 20:1: E402 module level import not at top of file
Line 145:80: E501 line too long (86 > 79 characters)
Line 153:80: E501 line too long (88 > 79 characters)
Line 226:80: E501 line too long (83 > 79 characters)
Line 237:80: E501 line too long (82 > 79 characters)
Line 243:80: E501 line too long (84 > 79 characters)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants