-
Notifications
You must be signed in to change notification settings - Fork 84
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
feat: flake8 and sonarcloud fixes #225
base: master
Are you sure you want to change the base?
Conversation
Remaining flake8 issues: File papirus/epd.py...
File papirus/text.py...
File papirus/textpos.py...
File projects/twitter_feed.py...
@ChristopherRush any idea on the ones for twitter_feed.py? As you wrote this, I thought you might be able to have a better idea |
For the papirus/text.py one I'm wondering Line 38 in 0023db4
|
For the papirus/textpos.py one, I'm thinking to just ignore this for now, and open a new issue on the repo to fix this another time. Thoughts? |
For the papirus/epd.py issues it looks like we can just use an r string: And split the line in two for the line length |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
I'm not a fan of camelCase (and it seems sonarcloud isn't either! 😂) However since this was only introduced in #144 I'm going to revert the changes I made on these and dismiss the code smells in sonarcloud 👍 |
No description provided.