-
Notifications
You must be signed in to change notification settings - Fork 4.1k
153 lines (136 loc) · 5.87 KB
/
spelling.yml
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
name: Check spelling
on:
pull_request:
push:
branches:
- main
jobs:
pyspelling:
runs-on: ubuntu-20.04
steps:
- name: Check for skip label and get changed files
id: check-files
uses: actions/github-script@v6
with:
script: |
let skipCheck = false;
let changedFiles = [];
if (context.eventName === 'pull_request') {
// Check for skip label
const { data: labels } = await github.rest.issues.listLabelsOnIssue({
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: context.issue.number
});
skipCheck = labels.some(label => label.name === 'skip-spell-check');
if (!skipCheck) {
// Get changed files in PR
const { data: files } = await github.rest.pulls.listFiles({
owner: context.repo.owner,
repo: context.repo.repo,
pull_number: context.issue.number
});
changedFiles = files
.filter(file => file.filename.match(/\.(py|rst|md)$/))
.map(file => file.filename);
}
} else {
// For push events, we'll still need to use git diff
// We'll handle this after checkout
}
core.setOutput('skip', skipCheck.toString());
core.setOutput('files', changedFiles.join('\n'));
core.setOutput('is-pr', (context.eventName === 'pull_request').toString());
- uses: actions/checkout@v4
if: steps.check-files.outputs.skip != 'true'
with:
fetch-depth: 0
- name: Get changed files for push event
if: |
steps.check-files.outputs.skip != 'true' &&
steps.check-files.outputs.is-pr != 'true'
id: push-files
run: |
CHANGED_FILES=$(git diff --name-only HEAD^..HEAD -- '*.py' '*.rst' '*.md')
echo "files<<EOF" >> $GITHUB_OUTPUT
echo "$CHANGED_FILES" >> $GITHUB_OUTPUT
echo "EOF" >> $GITHUB_OUTPUT
- name: Check if relevant files changed
if: steps.check-files.outputs.skip != 'true'
id: check
run: |
if [ "${{ steps.check-files.outputs.is-pr }}" == "true" ]; then
FILES="${{ steps.check-files.outputs.files }}"
else
FILES="${{ steps.push-files.outputs.files }}"
fi
if [ -z "$FILES" ]; then
echo "skip=true" >> $GITHUB_OUTPUT
echo "No relevant files changed (*.py, *.rst, *.md), skipping spell check"
else
echo "skip=false" >> $GITHUB_OUTPUT
echo "Found changed files to check:"
echo "$FILES"
fi
- uses: actions/setup-python@v4
if: |
steps.check-files.outputs.skip != 'true' &&
steps.check.outputs.skip != 'true'
with:
python-version: '3.9'
cache: 'pip'
- name: Install dependencies
if: |
steps.check-files.outputs.skip != 'true' &&
steps.check.outputs.skip != 'true'
run: |
pip install pyspelling
sudo apt-get install aspell aspell-en
- name: Run spell check on each file
id: spellcheck
if: |
steps.check-files.outputs.skip != 'true' &&
steps.check.outputs.skip != 'true'
run: |
if [ "${{ steps.check-files.outputs.is-pr }}" == "true" ]; then
mapfile -t FILES <<< "${{ steps.check-files.outputs.files }}"
else
mapfile -t FILES <<< "${{ steps.push-files.outputs.files }}"
fi
# Check each file individually
FINAL_EXIT_CODE=0
SPELLCHECK_LOG=""
for file in "${FILES[@]}"; do
if [ -n "$file" ]; then
echo "Checking spelling in $file"
python3 -c "import yaml; config = yaml.safe_load(open('.pyspelling.yml')); new_matrix = [matrix.copy() for matrix in config['matrix'] if (('python' in matrix['name'].lower() and '$file'.endswith('.py')) or ('rest' in matrix['name'].lower() and '$file'.endswith('.rst')) or ('markdown' in matrix['name'].lower() and '$file'.endswith('.md'))) and not matrix.update({'sources': ['$file']})]; config['matrix'] = new_matrix; yaml.dump(config, open('temp_config.yml', 'w'))"
if OUTPUT=$(pyspelling -c temp_config.yml 2>&1); then
echo "No spelling errors found in $file"
else
FINAL_EXIT_CODE=1
echo "Spelling errors found in $file:"
echo "$OUTPUT"
SPELLCHECK_LOG+="### $file\n$OUTPUT\n\n"
fi
fi
done
# Save the results to GITHUB_OUTPUT
echo "spell_failed=$FINAL_EXIT_CODE" >> $GITHUB_OUTPUT
echo "spell_log<<SPELLEOF" >> $GITHUB_OUTPUT
echo "$SPELLCHECK_LOG" >> $GITHUB_OUTPUT
echo "SPELLEOF" >> $GITHUB_OUTPUT
if [ $FINAL_EXIT_CODE -ne 0 ]; then
echo "Spell check failed! See above for details."
echo
echo "Here are a few tips:"
echo "- All PyTorch API objects must be in double backticks or use an intersphinx directive."
echo " Example: ``torch.nn``, :func:"
echo "- Consult en-wordlist.txt for spellings of some of the words."
echo " You can add a word to en-wordlist.txt if:"
echo " 1) It's a common abbreviation, like RNN."
echo " 2) It's a word widely accepted in the industry."
echo "- Please do not add words like 'dtype', 'torch.nn.Transformer' to pass spellcheck."
echo " Instead wrap it in double backticks or use an intersphinx directive."
echo
exit 1
fi