BaseTools/PatchCheck.py: Check CI Options Before Parsing

PatchCheck.py currently parses the CI options as the last
step it does before reporting results. This means that the
other checking logic cannot use any of the CI options that
are passed in.

This updates the order of operations to process CI options
before running other checks so that they can be used in
performing checks.

Signed-off-by: Oliver Smith-Denny <osde@microsoft.com>

1 files changed, 1 insertions, 1 deletions

diff --git a/BaseTools/Scripts/PatchCheck.py b/BaseTools/Scripts/PatchCheck.py
index 5c14840..937348d 100755
--- a/BaseTools/Scripts/PatchCheck.py
+++ b/BaseTools/Scripts/PatchCheck.py
@@ -117,13 +117,13 @@ class CommitMessageCheck:
 

         self.check_contributed_under()

         if not MergifyMerge:

+            self.check_ci_options_format()

             self.check_subject(updated_packages)

             self.check_signed_off_by()

             self.check_misc_signatures()

             self.check_overall_format()

             if not PatchCheckConf.ignore_change_id:

                 self.check_change_id_format()

-            self.check_ci_options_format()

         self.report_message_result()

 

     url = 'https://github.com/tianocore/tianocore.github.io/wiki/Commit-Message-Format'