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

New main script fixes #3

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

New main script fixes #3

wants to merge 8 commits into from

Conversation

plusreed
Copy link
Owner

No description provided.

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

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
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.
2 participants
@plusreed @pep8speaks