Skip to content

Commit

Permalink
Remove the unnecessary console object on Run
Browse files Browse the repository at this point in the history
The Run class doesn't need a rich.Console instance
for any reason. This change removes it and cleans up
the calls to run.

Signed-off-by: Eric Brown <eric.brown@securesauce.dev>
  • Loading branch information
ericwb committed Aug 27, 2024
1 parent 274c68f commit 73dbcfd
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion precli/cli/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -352,7 +352,7 @@ def main():
)

# Invoke the run
run = Run(enabled, disabled, artifacts, console, debug)
run = Run(enabled, disabled, artifacts, debug)
run.invoke()

# Render the results
Expand Down
3 changes: 0 additions & 3 deletions precli/core/run.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
from multiprocessing import Pool

from pygments import lexers
from rich.console import Console
from rich.progress import BarColumn
from rich.progress import MofNCompleteColumn
from rich.progress import Progress
Expand Down Expand Up @@ -116,13 +115,11 @@ def __init__(
enabled: list[str],
disabled: list[str],
artifacts: list[Artifact],
console: Console,
debug: int,
):
self._enabled = enabled
self._disabled = disabled
self._artifacts = artifacts
self._console = console
self._init_logger(debug)
self._start_time = None
self._end_time = None
Expand Down

0 comments on commit 73dbcfd

Please sign in to comment.