Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add "docker-compose.yml" to "ignorePaths" array in cspell.json #5849

Closed
4 of 6 tasks
Tracked by #5450
freaky4wrld opened this issue Nov 5, 2023 · 3 comments · Fixed by #6014
Closed
4 of 6 tasks
Tracked by #5450

Add "docker-compose.yml" to "ignorePaths" array in cspell.json #5849

freaky4wrld opened this issue Nov 5, 2023 · 3 comments · Fixed by #6014
Assignees
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue feature: spelling role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less

Comments

@freaky4wrld
Copy link
Member

freaky4wrld commented Nov 5, 2023

Overview

We need to populate the spell check configuration file with file paths that should be excluded from spell checking,

Action Items

  • You must use VS Code as your local text editor and use the Code Spell Checker Extension for this issue, The recommended installation method is to install Code Spell Checker directly from the VS Code text editor1. The extension can also be installed via the VS Code Marketplace website2
  • In VSCode, navigate to 'cspell.json`
  • Add the path "docker-compose.yml" to the end of the array "ignorePaths"
  • Save 'cspell.json` and test the update by confirming that files in the specified path are not spell checked.

Merge Team

Resources/Instructions

Introduction to JSON
Code Spell Checker
This issue is part of #5450

Footnotes

  1. Installing Code Spell Checker from the VS Code text editor

  2. Code Spell Checker - VS Marketplace

@freaky4wrld freaky4wrld added good first issue Good for newcomers Feature: Administrative Administrative chores etc. role: front end Tasks for front end developers role: back end/devOps Tasks for back-end developers ready for dev lead Issues that tech leads or merge team members need to follow up on size: 0.25pt Can be done in 0.5 to 1.5 hours labels Nov 5, 2023
@ExperimentsInHonesty ExperimentsInHonesty added feature: spelling and removed Feature: Administrative Administrative chores etc. labels Nov 8, 2023
@ExperimentsInHonesty ExperimentsInHonesty added this to the x. Technical debt milestone Nov 8, 2023
@roslynwythe roslynwythe added Complexity: Small Take this type of issues after the successful merge of your second good first issue size: 0.5pt Can be done in 3 hours or less and removed ready for dev lead Issues that tech leads or merge team members need to follow up on good first issue Good for newcomers size: 0.25pt Can be done in 0.5 to 1.5 hours labels Nov 17, 2023
@dimiprousalis dimiprousalis self-assigned this Dec 13, 2023
Copy link

Hi @dimiprousalis, thank you for taking up this issue! Hfla appreciates you :)

Do let fellow developers know about your:-
i. Availability: (When are you available to work on the issue/answer questions other programmers might have about your issue?)
ii. ETA: (When do you expect this issue to be completed?)

You're awesome!

P.S. - You may not take up another issue until this issue gets merged (or closed). Thanks again :)

@dimiprousalis
Copy link
Member

It looks like this is no longer a valid issue? In cspell.json the path ".github" is already included in the array "ignorePaths"

@roslynwythe roslynwythe changed the title Add ".github" to "ignorePaths" array in cspell.json Add "docker-compose.yml" to "ignorePaths" array in cspell.json Dec 15, 2023
@roslynwythe
Copy link
Member

Hi @dimiprousalis I apologize for the error. I had put ".github" in the list as part of the testing for cspell, and so it should have not been included in an issue. I edited the current issue to specify a new path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue feature: spelling role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less
Projects
Development

Successfully merging a pull request may close this issue.

4 participants