Skip to content
This repository was archived by the owner on Apr 6, 2022. It is now read-only.

Commit fc5a420

Browse files
committed
Revert "check_compliance.py: Make license check informational"
This reverts commit 747795b. We want the license check to error out. Signed-off-by: Kumar Gala <[email protected]>
1 parent 5b98805 commit fc5a420

File tree

1 file changed

+6
-9
lines changed

1 file changed

+6
-9
lines changed

scripts/check_compliance.py

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -876,15 +876,12 @@ def run(self):
876876
report += ("* {} missing copyright.\n".format(orig_path))
877877

878878
if report != "":
879-
self.add_info("""
880-
This is just an FYI for maintainers. In most cases you do not need to do
881-
anything here, especially if the files reported below are going into ext/ and
882-
the license was approved for inclusion into ext/ already. Fix any missing
883-
license/copyright issues.
884-
885-
A compact format is encouraged, to avoid too much boilerplate in files.
886-
887-
""" + report)
879+
self.add_failure("""
880+
In most cases you do not need to do anything here, especially if the files
881+
reported below are going into ext/ and if license was approved for inclusion
882+
into ext/ already. Fix any missing license/copyright issues. The license
883+
exception if a JFYI for the maintainers and can be overriden when merging the
884+
pull request.\n\n""" + report)
888885

889886

890887
class Identity(ComplianceTest):

0 commit comments

Comments
 (0)