check_readme_links: remove throwing of exception before exit

Reraising the exception before exiting was intended to help troubleshoot,
but turned out to be more confusing than helpful as it might look like the script was failing
This commit is contained in:
Marius Vikhammer 2021-04-26 15:36:30 +08:00
parent 7defd291a4
commit 4bf7b16358

View File

@ -22,6 +22,7 @@ import concurrent.futures
import os
import os.path
import re
import sys
import urllib.error
import urllib.request
from collections import defaultdict, namedtuple
@ -148,8 +149,8 @@ def check_readme_links(args):
print('Found {} errors:'.format(len(errors)))
for e in errors:
print(e)
if errors:
raise e
return 1 if len(errors) > 0 else 0
if __name__ == '__main__':
@ -158,4 +159,4 @@ if __name__ == '__main__':
parser.add_argument('--skip-weburl', '-w', action='store_true', help='Skip checking of web URLs, only check links to local files')
args = parser.parse_args()
check_readme_links(args)
sys.exit(check_readme_links(args))