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 more software dev related words to the default dicts #3499

Open
spotlesscoder opened this issue Aug 23, 2024 · 3 comments
Open

Add more software dev related words to the default dicts #3499

spotlesscoder opened this issue Aug 23, 2024 · 3 comments

Comments

@spotlesscoder
Copy link
Collaborator

spotlesscoder commented Aug 23, 2024

Redis Database

  • rediss (probably only whitelist when followed by :// for the redis secure protocol handler - we do not want to have mistyped "Redis" with duplicate s in other locations than rediss:// URLs)

Vs Code

  • devcontainer

Azure

  • keyvault
  • azurecr (Azure Container Registry)
  • azurerm (Azure Resource Manager)
  • azurewebsites (azurewebsites.net domain)
  • swedencentral (Sweden Central Location in Azure)

Terraform

  • tfvars (Terraform Variables File)
  • tfstate (Terraform State)
@Jason3S
Copy link
Collaborator

Jason3S commented Aug 23, 2024

@spotlesscoder,

Thank you for the suggestions! If you are up for it, we could use your help. It is very easy to add them to the word lists. I have added links below. If you follow the links and click the edit button, (after forking) add the words to the end of the files, they will get auto sorted after the PRs are landed. No need to download or build anything.

Redis Database

  • rediss (probably only whitelist when followed by :// for the redis secure protocol handler - we do not want to have mistyped "Redis" with duplicate s in other locations than rediss:// URLs)

This one is not so easy. There isn't a way to add a qualifier. Most URLs should already be ignored. Do you have some examples? It might be necessary to expand the URL detection.

Vs Code

  • devcontainer

Please add this to:
https://github.com/streetsidesoftware/cspell-dicts/blob/main/dictionaries/software-terms/src/coding-terms.txt

It is common enough.

Azure

  • keyvault
  • azurecr (Azure Container Registry)
  • azurerm (Azure Resource Manager)
  • azurewebsites (azurewebsites.net domain)
  • swedencentral (Sweden Central Location in Azure)

There are a couple of options here:

Terraform

  • tfvars (Terraform Variables File)
  • tfstate (Terraform State)

Please add them to: https://github.com/streetsidesoftware/cspell-dicts/blob/main/dictionaries/terraform/src/terraform.txt

@spotlesscoder
Copy link
Collaborator Author

rediss://localhost
rediss://username:[email protected]/1

should be examples for valid URLs

spotlesscoder added a commit to spotlesscoder/cspell-dicts that referenced this issue Aug 26, 2024
Jason3S added a commit that referenced this issue Aug 26, 2024
@spotlesscoder
Copy link
Collaborator Author

sorry it took me so long - was very busy
what should we do with the rediss protocol handler now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants