Skip to content

Commit ff853e1

Browse files
authored
Merge pull request hossein-zare#698 from taeh98/values-error
chore: add ci checks, order fields, fix types, and simplify logic
2 parents bd36006 + 05297d0 commit ff853e1

File tree

6 files changed

+560
-500
lines changed

6 files changed

+560
-500
lines changed

.github/workflows/ci.yml

Lines changed: 22 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,5 +85,26 @@ jobs:
8585
- run: npm outdated
8686
- run: npx npm-check-updates
8787

88+
typos_spell_check:
89+
name: Spell Check with Typos
90+
runs-on: ubuntu-latest
91+
92+
steps:
93+
- uses: actions/checkout@v2
94+
- name: Check spelling with typos
95+
uses: crate-ci/typos@master
96+
with:
97+
config: .typos.toml
98+
99+
commitlint:
100+
name: Check the commit conforms to the Conventional Commits specification with commitlint
101+
runs-on: ubuntu-latest
102+
103+
steps:
104+
- uses: actions/checkout@v3
105+
- uses: wagoid/commitlint-github-action@v5
106+
107+
# TODO: check commits squashed into one (i.e. all changes in a PR are in one commit after last one in primary branch)
108+
# TODO: use release-please to increment version number, update changelog, and publish releases automatically if CI passes
88109
# TODO: check project builds/compiles correctly
89-
# TODO: add tests
110+
# TODO: add tests including coverage

.typos.toml

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
# docs at: https://github.com/crate-ci/typos/blob/master/docs/reference.md
2+
3+
files.extend-exclude = [
4+
".typos.toml"
5+
]
6+
7+
default.extend-ignore-re = [
8+
# false positives
9+
"alguns itens selecionados",
10+
"elementi sono stati selezionati",
11+
"Nada a ser mostrado",
12+
"prevProps",
13+
"prevState",
14+
]
15+
16+
default.locale = "en-us"
17+
18+
# note: add false negatives with corrections in default.extend-identifiers or default.extend-words
19+
# todo: check mistakes over multiple words, e.g. "the the" instead of "the"

0 commit comments

Comments
 (0)