Skip to content

Commit f36fd46

Browse files
Simekmeta-codesync[bot]
authored andcommitted
do not spellcheck vendored scripts (#55020)
Summary: While running `yarn spellcheck` I have spotted that scripts located in `/vendor` directory are checked, which lead to a lot of warnings in the output. This PR adds an exclusion for the `/vendor` path in the spellcheck script which leads to way smaller warnings set and saves ~10s of runtime on my machine. ## Changelog: [INTERNAL] [FIXED] - Add `/vendor` to excluded paths for spellcheck script. Pull Request resolved: #55020 Test Plan: Running `yarn spellcheck` locally. Reviewed By: cipolleschi Differential Revision: D90104078 Pulled By: cortinico fbshipit-source-id: 9c16a71616a13db1c217b917a498ff8f6a462799
1 parent 0241b59 commit f36fd46

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

.github/workflow-scripts/analyze_scripts.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ if [ -x "$(command -v shellcheck)" ]; then
1616
-type f \
1717
-not -path "*node_modules*" \
1818
-not -path "*third-party*" \
19+
-not -path "*vendor*" \
1920
-name '*.sh' \
2021
-exec sh -c 'shellcheck "$1"' -- {} \;
2122

0 commit comments

Comments
 (0)