Skip to content

Commit b492cfe

Browse files
authored
Merge pull request #645 from github/copilot/fix-spelling-grammar-issues
Add codespell CI workflow for automated spelling checks
2 parents f1367eb + 20818f2 commit b492cfe

3 files changed

Lines changed: 39 additions & 1 deletion

File tree

.codespellrc

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
[codespell]
2+
# Ignore intentional misspellings used as examples and technical terms
3+
# numer - intentional example typo in add-educational-comments.prompt.md
4+
# wit - proper technical term/name (sardonic wit, Gilfoyle character trait)
5+
# aks - Azure Kubernetes Service (AKS) abbreviation
6+
# edn - Extensible Data Notation (Clojure data format)
7+
# ser - serialization abbreviation
8+
# ois - ObjectInputStream abbreviation in Java
9+
# gir - valid abbreviation/technical term
10+
# rouge - Rouge is a syntax highlighter (not "rogue")
11+
# categor - TypeScript template literal in website/src/scripts/pages/skills.ts:70 (categor${...length > 1 ? "ies" : "y"})
12+
# aline - proper name (Aline Ávila, contributor)
13+
# ative - part of "Declarative Agents" in TypeSpec M365 Copilot documentation (collections/typespec-m365-copilot.collection.md)
14+
ignore-words-list = numer,wit,aks,edn,ser,ois,gir,rouge,categor,aline,ative,afterall,deques
15+
# Skip certain files and directories
16+
skip = .git,node_modules,package-lock.json,*.lock,website/build,website/.docusaurus

.github/copilot-instructions.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ The following instructions are only to be applied when performing a code review.
2525
- [ ] The instruction has a `description` field.
2626
- [ ] The `description` field is not empty.
2727
- [ ] The file name is lower case, with words separated by hyphens.
28-
- [ ] The instruction has an `applyTo` field that specifies the file or files to which the instructions apply. If they wish to specify multiple file paths they should formated like `'**.js, **.ts'`.
28+
- [ ] The instruction has an `applyTo` field that specifies the file or files to which the instructions apply. If they wish to specify multiple file paths they should formatted like `'**.js, **.ts'`.
2929

3030
## Agent file guide
3131

.github/workflows/codespell.yml

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
name: Check Spelling
2+
3+
on:
4+
push:
5+
branches: [main]
6+
pull_request:
7+
branches: [main]
8+
9+
permissions:
10+
contents: read
11+
12+
jobs:
13+
codespell:
14+
runs-on: ubuntu-latest
15+
steps:
16+
- uses: actions/checkout@v4
17+
18+
- name: Check spelling with codespell
19+
uses: codespell-project/actions-codespell@v2
20+
with:
21+
check_filenames: true
22+
check_hidden: false

0 commit comments

Comments
 (0)