Ignore `google-apps-scripts/` in spell checker

hackforla / website

Hack for LA's website

https://www.hackforla.org

GNU General Public License v2.0

286 stars 700 forks source link

Ignore `google-apps-scripts/` in spell checker #6788

Closed jchue closed 2 weeks ago

jchue commented 2 weeks ago

Fixes #5845

What changes did you make?

Add google-apps-scripts/ directory to ignorePaths array in cspell.json

Why did you make the changes (we will use this info to test)?

To exclude google-apps-scripts/ from spell check

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

No visual changes

github-actions[bot] commented 2 weeks ago

Want to review this pull request? Take a look at this documentation for a step by step guide!

From your project repository, check out a new branch and test the changes.

git checkout -b jchue-ignore-google-apps-scripts-in-cspell-5845 gh-pages
git pull https://github.com/jchue/hackforla-website.git ignore-google-apps-scripts-in-cspell-5845

Note that CONTRIBUTING.md cannot previewed locally; rather it should be previewed at this URL:

https://github.com/jchue/website/blob/ignore-google-apps-scripts-in-cspell-5845/CONTRIBUTING.md

freaky4wrld commented 2 weeks ago

@jchue under the Screenshots of Proposed Changes Of The Website heading remove all it's content and mention No visual changes as this issue has no visual changes to the website.

jchue commented 2 weeks ago

@freaky4wrld Updated!

urvishp13 commented 2 weeks ago

Review ETA: 3pm 5/2/24 Availability: 12pm - 3pm 5/2/24

patelbansi3009 commented 2 weeks ago

Review ETA: EOD of 05/01/24 Availability: Full Day 05/01/24

patelbansi3009 commented 2 weeks ago

Great work @jchue!

Used the correct branch.
Accurately linked the issue.
Code changes are made as requested in the issue.
There are no visual changes on the website.

Keep up the good work!