Merge pull request #545 from lgeiger/list-comprehensions
Prefer list comprehensions over list.append()
This commit is contained in:
@@ -63,10 +63,7 @@ class NodeManager(object):
|
|||||||
def setup_nodes(self, putevent):
|
def setup_nodes(self, putevent):
|
||||||
self.config.hook.pytest_xdist_setupnodes(config=self.config, specs=self.specs)
|
self.config.hook.pytest_xdist_setupnodes(config=self.config, specs=self.specs)
|
||||||
self.trace("setting up nodes")
|
self.trace("setting up nodes")
|
||||||
nodes = []
|
return [self.setup_node(spec, putevent) for spec in self.specs]
|
||||||
for spec in self.specs:
|
|
||||||
nodes.append(self.setup_node(spec, putevent))
|
|
||||||
return nodes
|
|
||||||
|
|
||||||
def setup_node(self, spec, putevent):
|
def setup_node(self, spec, putevent):
|
||||||
gw = self.group.makegateway(spec)
|
gw = self.group.makegateway(spec)
|
||||||
@@ -158,11 +155,10 @@ class HostRSync(execnet.RSync):
|
|||||||
|
|
||||||
def __init__(self, sourcedir, *args, **kwargs):
|
def __init__(self, sourcedir, *args, **kwargs):
|
||||||
self._synced = {}
|
self._synced = {}
|
||||||
self._ignores = []
|
|
||||||
ignores = kwargs.pop("ignores", None) or []
|
ignores = kwargs.pop("ignores", None) or []
|
||||||
for x in ignores:
|
self._ignores = [
|
||||||
x = getattr(x, "strpath", x)
|
re.compile(fnmatch.translate(getattr(x, "strpath", x))) for x in ignores
|
||||||
self._ignores.append(re.compile(fnmatch.translate(x)))
|
]
|
||||||
super(HostRSync, self).__init__(sourcedir=sourcedir, **kwargs)
|
super(HostRSync, self).__init__(sourcedir=sourcedir, **kwargs)
|
||||||
|
|
||||||
def filter(self, path):
|
def filter(self, path):
|
||||||
|
|||||||
Reference in New Issue
Block a user