Skip to content

Commit

Permalink
Apply black suggestions as of 2024-08-01
Browse files Browse the repository at this point in the history
  • Loading branch information
okurz committed Aug 1, 2024
1 parent f8e7fcc commit 4511e92
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 14 deletions.
1 change: 0 additions & 1 deletion openqa_review/browser.py
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,6 @@ def filename_to_url(name):


class Browser(object):

"""download relative or absolute url and return soup."""

def __init__(self, args, root_url, auth=None, headers={}, api_key=None):
Expand Down
22 changes: 11 additions & 11 deletions openqa_review/openqa_review.py
Original file line number Diff line number Diff line change
Expand Up @@ -689,7 +689,6 @@ def _parse_issue_timestamp(timestamp):


class Issue(object):

"""Issue with extra status info from issue tracker."""

def __init__(self, bugref, bugref_href, query_issue_status=False, progress_browser=None, bugzilla_browser=None):
Expand Down Expand Up @@ -887,7 +886,6 @@ def _format_all_urls_using_markdown(string):


class IssueEntry(object):

"""List of failed test scenarios with corresponding bug."""

def __init__(self, args, root_url, failures, test_browser=None, bug=None):
Expand Down Expand Up @@ -1163,13 +1161,17 @@ def __str__(self):
"status_badge": status_badge_str[self.status_badge],
# everything that is 'NEW_ISSUE' should be product issue but if tests have changed content, then
# probably openqa issues For now we can just not easily decide unless we use the 'bugrefs' mode
"new_openqa_issues": ""
if abbrev
else issue_listing("**New openQA-issues:**", self.issues["new"]["openqa"], self.args.show_empty),
"existing_openqa_issues": ""
if abbrev
else issue_listing(
"**Existing openQA-issues:**", self.issues["existing"]["openqa"], self.args.show_empty
"new_openqa_issues": (
""
if abbrev
else issue_listing("**New openQA-issues:**", self.issues["new"]["openqa"], self.args.show_empty)
),
"existing_openqa_issues": (
""
if abbrev
else issue_listing(
"**Existing openQA-issues:**", self.issues["existing"]["openqa"], self.args.show_empty
)
),
"new_product_issues": issue_listing(
"**New Product bugs:**", self.issues["new"]["product"], self.args.show_empty
Expand All @@ -1184,7 +1186,6 @@ def __str__(self):


class ProductReport(object):

"""Read overview page of one job group and generate a report for the product."""

def __init__(self, browser, job_group_url, root_url, args):
Expand Down Expand Up @@ -1528,7 +1529,6 @@ def _pgroup_prefix(group):


class Report(object):

"""openQA review report."""

def __init__(self, browser, args, root_url, job_groups):
Expand Down
2 changes: 0 additions & 2 deletions openqa_review/tumblesle_release.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,14 +68,12 @@ def scenario(job):


class UnsupportedRsyncArgsError(Exception):

"""Unsupported rsync arguments where used."""

pass


class TumblesleRelease(object):

"""Check for releasable builds and release them as TumbleSLE if they are at least as good as the current one."""

def __init__(self, args):
Expand Down

0 comments on commit 4511e92

Please sign in to comment.