Use travis ci to moderate pylint styles.

Author: Taaku18

.lint.py

@@ -0,0 +1,17 @@
+import sys
+from os import listdir
+from os.path import join
+
+from pylint.lint import Run
+
+THRESHOLD = 9.75
+
+cogs = [join('cogs', c) for c in listdir('cogs') if c.endswith('.py')]
+core = [join('core', c) for c in listdir('core') if c.endswith('.py')]
+
+results = Run(['bot.py', *cogs, *core],
+              do_exit=False)
+
+score = results.linter.stats['global_note']
+if score <= THRESHOLD:
+    sys.exit(1)

.pylintrc

@@ -121,7 +121,7 @@ never-returning-functions=sys.exit
 
 # Format style used to check logging format string. `old` means using %
 # formatting, while `new` is for `{}` formatting.
-logging-format-style=old
+logging-format-style=new
 
 # Logging modules to check that the string format arguments are in logging
 # function parameter format.
@@ -152,8 +152,8 @@ spelling-store-unknown-words=no
 
 # List of note tags to take in consideration, separated by a comma.
 notes=FIXME,
-      XXX,
-      TODO
+      TODO,
+      BUG
 
 
 [TYPECHECK]
@@ -351,11 +351,15 @@ good-names=i,
            j,
            k,
            ex,
-           Run,
-           _
+           _,
+           v,
+           id,
+           db,
+           f,
+           dt
 
 # Include a hint for the correct naming format with invalid-name.
-include-naming-hint=no
+include-naming-hint=yes
 
 # Naming style matching correct inline iteration names.
 inlinevar-naming-style=any

.travis.yml

@@ -0,0 +1,12 @@
+language: python
+
+matrix:
+  include:
+  - python: '3.7'
+    dist: xenial
+
+install:
+  - pip install -r requirements.txt
+  - pip install pylint
+
+script: python .lint.py

bot.py

@@ -477,13 +477,13 @@ async def on_message_edit(self, before, after):
                                                     after.content)
                         break
 
-    async def on_command_error(self, ctx, error):
-        if isinstance(error, (commands.MissingRequiredArgument,
-                              commands.UserInputError)):
-            await ctx.invoke(self.get_command('help'),
-                             command=str(ctx.command))
+    async def on_command_error(self, context, exception):
+        if isinstance(exception, (commands.MissingRequiredArgument,
+                                  commands.UserInputError)):
+            await context.invoke(self.get_command('help'),
+                                 command=str(context.command))
         else:
-            raise error
+            raise exception
 
     @staticmethod
     def overwrites(ctx):
@@ -533,10 +533,10 @@ async def validate_api_token(self):
     async def validate_database_connection(self):
         try:
             await self.db.command('buildinfo')
-        except Exception as e:
+        except Exception as exc:
             print(Fore.RED, end='')
             print('Something went wrong while connecting to the database.')
-            print(type(e).__name__, e, sep=': ')
+            print(type(exc).__name__, exc, sep=': ')
             return await self.logout()
         else:
             print(Style.RESET_ALL + Fore.CYAN +

requirements.txt

@@ -13,5 +13,4 @@ sanic==18.12.0
 dnspython==1.16.0
 aiohttp==3.4.4
 pymongo==3.7.2
-uvloop==0.12.0
-pylint==2.2.2
+uvloop==0.12.0