Skip to content

Commit

Permalink
Cleans up flake8 rules, per review
Browse files Browse the repository at this point in the history
@dachary pointed out a few touch-ups. In particular, the change to
`ossec_urls.py` was a nice catch, in that the "noqa" line was ignoring
*all* flake8 violations, not merely the specific one mentioned. @dachary
further pointed out that it wasn't E405, but rather F405, that should
have been permitted.
  • Loading branch information
Conor Schaefer committed Jan 29, 2018
1 parent 454b867 commit 1a9ffe4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ def playbook_on_stats(self, stats):

total_seconds = sum([x[1] for x in self.stats.items()])
print("\nPlaybook finished: {0}, {1} total tasks."
"{2} elapsed. \n".format(
" {2} elapsed. \n".format(
time.asctime(),
len(self.stats.items()),
datetime.timedelta(seconds=(int(total_seconds)))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ def parse_checksums(self):


def main():
module = AnsibleModule( # noqa E405
module = AnsibleModule( # noqa: F405
argument_spec=dict(
ossec_version=dict(default="2.8.2"),
),
Expand Down

0 comments on commit 1a9ffe4

Please sign in to comment.