Skip to content

Commit

Permalink
Merge pull request #729 from douglasjacobsen/exit-code-prints
Browse files Browse the repository at this point in the history
Remove unneeded prints from exit-code modifier
  • Loading branch information
linsword13 authored Oct 30, 2024
2 parents eacd650 + f0eda27 commit 74d55b0
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions var/ramble/repos/builtin/modifiers/exit-code/modifier.py
Original file line number Diff line number Diff line change
Expand Up @@ -144,8 +144,6 @@ def _examine_all_exit_codes(self, workspace, app_inst=None):
app_inst.expander.expand_var("{exit_code_log}"), workspace
)

logger.all_msg(f" Exit code log: {log_file}")

exit_regex = re.compile(self.exit_code_regex)
final_regex = re.compile("Final exit code: (?P<exit_code>[0-9]+)")

Expand All @@ -155,11 +153,9 @@ def _examine_all_exit_codes(self, workspace, app_inst=None):
if os.path.exists(log_file):
with open(log_file) as f:
for line in f.readlines():
logger.all_msg(f" Exit line: {line}")
m = exit_regex.match(line)

if m:
logger.all_msg(f' Match: {m.group("exit_code")}')
max_code = max(max_code, int(m.group("exit_code")))

if final_regex.match(line):
Expand Down

0 comments on commit 74d55b0

Please sign in to comment.