Skip to content

Commit

Permalink
Merge pull request #10 from antmicro/wsip/fix_empty_output
Browse files Browse the repository at this point in the history
Fix missing output from action.py
  • Loading branch information
tgorochowik committed Sep 22, 2021
2 parents c6eea69 + 8f940fd commit c761095
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions action.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,9 +92,7 @@ def main(conf_file, extra_opts, exclude_paths, log_file, patch, path):
extra_opts = []

if patch:
patch = ["--autofix=yes", "--autofix_output_file=" + patch]
# use this for newer version of Verible:
#patch = ["--autofix=patch", "--autofix_output_file=" + patch]
patch = ["--autofix=patch", "--autofix_output_file=" + patch]
else:
patch = []

Expand All @@ -116,7 +114,7 @@ def main(conf_file, extra_opts, exclude_paths, log_file, patch, path):
print("Running " + " ".join(command) + "\n\n")
verible_linted = subprocess.run(command, capture_output=True)

issues = verible_linted.stdout.decode("utf-8")
issues = verible_linted.stderr.decode("utf-8")
log_raw(issues, log_file if log_file else 'verible-verilog-lint.log')

exit(issues != "")
Expand Down

0 comments on commit c761095

Please sign in to comment.