Skip to content
This repository was archived by the owner on Mar 11, 2019. It is now read-only.

New main script fixes #3

Open
plusreed wants to merge 8 commits into
base: master
Choose a base branch
from
Open

New main script fixes #3

plusreed wants to merge 8 commits into from

Conversation

@plusreed
Copy link
Owner

@plusreed plusreed commented May 18, 2017

No description provided.

@plusreed
Fix getLogger bug
aee0fcc 
No quotation marks were around 'discord' which caused unintended behavior.
@pep8speaks
Copy link

pep8speaks commented May 18, 2017

Hello @plusreed! Thanks for submitting the PR.

  • In the file main3.py, following are the PEP8 issues :

Line 3:16: E401 multiple imports on one line
Line 36:80: E501 line too long (98 > 79 characters)
Line 42:80: E501 line too long (108 > 79 characters)
Line 44:80: E501 line too long (100 > 79 characters)
Line 48:80: E501 line too long (84 > 79 characters)
Line 75:80: E501 line too long (95 > 79 characters)
Line 97:80: E501 line too long (96 > 79 characters)

plusreed added 7 commits May 18, 2017 11:19
@plusreed
Fix load_extension error
cda01a2 
Accidentally misspelled load_extension as load_extention, so no cogs were loaded
@plusreed
Adding CircleCI file
d232f70
@plusreed
Adding test command
d62e40f 
More just playing around at this point
@plusreed
Fixing possible bug
c77d18d
@plusreed
Update circle.yml
8711b96
@plusreed
Update circle.yml
9690d5a
@plusreed
Update circle.yml
80e2e45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@plusreed @pep8speaks