michaelh/zephyr
1
0
Fork 0
Code Releases Activity

doc: application.py: Make functions that don't use 'self' static

Browse source 
Fixes this pylint warning:

    R0201: Method could be a function (no-self-use)

Another option would be to turn them into regular functions, but that'd
be a bigger change.

Fixing pylint warnings for a CI check.

Signed-off-by: Ulf Magnusson <Ulf.Magnusson@nordicsemi.no>
This commit is contained in:
Ulf Magnusson 2019-09-04 17:04:56 +02:00 • committed by Anas Nashif
commit b558956bd7
1 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
doc/extensions/zephyr/application.py
View file

@ -285,7 +285,8 @@ class ZephyrAppCommandsDirective(Directive):
return content return content
def _mkdir(self, mkdir, build_dir, host_os, skip_config): @staticmethod
def _mkdir(mkdir, build_dir, host_os, skip_config):
content = [] content = []
if skip_config: if skip_config:
content.append("# If you already made a build directory ({}) and ran cmake, just 'cd {}' instead.".format(build_dir, build_dir)) # noqa: E501 content.append("# If you already made a build directory ({}) and ran cmake, just 'cd {}' instead.".format(build_dir, build_dir)) # noqa: E501
@ -298,7 +299,8 @@ class ZephyrAppCommandsDirective(Directive):
content.append('mkdir {} & cd {}'.format(build_dir, build_dir)) content.append('mkdir {} & cd {}'.format(build_dir, build_dir))
return content return content
def _cmake_args(self, **kwargs): @staticmethod
def _cmake_args(**kwargs):
board = kwargs['board'] board = kwargs['board']
shield = kwargs['shield'] shield = kwargs['shield']
conf = kwargs['conf'] conf = kwargs['conf']