From c1215973762ab274805b37bb3fc1770f2c74d327 Mon Sep 17 00:00:00 2001
From: VictoriousRaptor <10308169+VictoriousRaptor@users.noreply.github.com>
Date: Sat, 31 Dec 2022 23:37:40 +0800
Subject: [PATCH] Fix sarif (#1752)

* Fix SARIF

* only check changed files

* mask http/https

* add newline at eof

* Add dicts

* fix http regex

* fix http regex
---
 .github/actions/spelling/patterns.txt | 4 ++++
 .github/workflows/spelling.yml        | 6 ++++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/.github/actions/spelling/patterns.txt b/.github/actions/spelling/patterns.txt
index 73095a638..9db69597e 100644
--- a/.github/actions/spelling/patterns.txt
+++ b/.github/actions/spelling/patterns.txt
@@ -108,3 +108,7 @@
 # Localization keys
 #x:Key="[^"]+"
 #{DynamicResource [^"]+}
+
+#http/https
+(?:\b(?:https?|ftp|file)://)[-A-Za-z0-9+&@#/%?=~_|!:,.;]+[-A-Za-z0-9+&@#/%=~_|]
+
diff --git a/.github/workflows/spelling.yml b/.github/workflows/spelling.yml
index 125eee53a..7a7a48bcd 100644
--- a/.github/workflows/spelling.yml
+++ b/.github/workflows/spelling.yml
@@ -82,10 +82,10 @@ jobs:
         post_comment: 0
         use_magic_file: 1
         experimental_apply_changes_via_bot: 1
-        use_sarif: true # to show in pr page
+        use_sarif: 0 # to show in pr page
         extra_dictionary_limit: 10
         check_commit_messages: commits title description
-        only_check_changed_files: false
+        only_check_changed_files: 1
         check_extra_dictionaries: ''
         quit_without_error: true
         extra_dictionaries:
@@ -96,6 +96,8 @@ jobs:
           cspell:csharp/csharp.txt
           cspell:dotnet/dotnet.txt
           cspell:python/src/python/python-lib.txt
+          cspell:aws/aws.txt
+          cspell:companies/src/companies.txt
         warnings:
           binary-file,deprecated-feature,large-file,limited-references,noisy-file,non-alpha-in-dictionary,unexpected-line-ending,whitespace-in-dictionary,minified-file,unsupported-configuration,unrecognized-spelling,no-newline-at-eof