lgtm: fix Mismatch between signature and use of an overridden method
by renaming process_kwargs to process_kwargs_base for the base target class.
This commit is contained in:
parent
e6c15b27e6
commit
268d59516f
|
@ -411,7 +411,7 @@ a hard error in the future.''' % name)
|
||||||
return self.construct_id_from_path(
|
return self.construct_id_from_path(
|
||||||
self.subdir, self.name, self.type_suffix())
|
self.subdir, self.name, self.type_suffix())
|
||||||
|
|
||||||
def process_kwargs(self, kwargs):
|
def process_kwargs_base(self, kwargs):
|
||||||
if 'build_by_default' in kwargs:
|
if 'build_by_default' in kwargs:
|
||||||
self.build_by_default = kwargs['build_by_default']
|
self.build_by_default = kwargs['build_by_default']
|
||||||
if not isinstance(self.build_by_default, bool):
|
if not isinstance(self.build_by_default, bool):
|
||||||
|
@ -789,7 +789,7 @@ class BuildTarget(Target):
|
||||||
return self.install_mode
|
return self.install_mode
|
||||||
|
|
||||||
def process_kwargs(self, kwargs, environment):
|
def process_kwargs(self, kwargs, environment):
|
||||||
super().process_kwargs(kwargs)
|
self.process_kwargs_base(kwargs)
|
||||||
self.copy_kwargs(kwargs)
|
self.copy_kwargs(kwargs)
|
||||||
kwargs.get('modules', [])
|
kwargs.get('modules', [])
|
||||||
self.need_install = kwargs.get('install', self.need_install)
|
self.need_install = kwargs.get('install', self.need_install)
|
||||||
|
@ -2068,7 +2068,7 @@ class CustomTarget(Target):
|
||||||
return final_cmd
|
return final_cmd
|
||||||
|
|
||||||
def process_kwargs(self, kwargs, backend):
|
def process_kwargs(self, kwargs, backend):
|
||||||
super().process_kwargs(kwargs)
|
self.process_kwargs_base(kwargs)
|
||||||
self.sources = extract_as_list(kwargs, 'input', unholder=True)
|
self.sources = extract_as_list(kwargs, 'input', unholder=True)
|
||||||
if 'output' not in kwargs:
|
if 'output' not in kwargs:
|
||||||
raise InvalidArguments('Missing keyword argument "output".')
|
raise InvalidArguments('Missing keyword argument "output".')
|
||||||
|
@ -2252,6 +2252,9 @@ class RunTarget(Target):
|
||||||
repr_str = "<{0} {1}: {2}>"
|
repr_str = "<{0} {1}: {2}>"
|
||||||
return repr_str.format(self.__class__.__name__, self.get_id(), self.command)
|
return repr_str.format(self.__class__.__name__, self.get_id(), self.command)
|
||||||
|
|
||||||
|
def process_kwargs(self, kwargs):
|
||||||
|
return self.process_kwargs_base(kwargs)
|
||||||
|
|
||||||
def get_dependencies(self):
|
def get_dependencies(self):
|
||||||
return self.dependencies
|
return self.dependencies
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue