Skip to content

Conversation

@felangel
Copy link
Contributor

Description

  • feat: spell check workflow

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • πŸ› οΈ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • βœ… Build configuration change
  • πŸ“ Documentation
  • πŸ—‘οΈ Chore

@felangel felangel added the feature A new feature or request label Oct 11, 2022
@felangel felangel self-assigned this Oct 11, 2022
@felangel felangel marked this pull request as ready for review October 11, 2022 03:10
@felangel felangel requested a review from erickzanardo October 11, 2022 03:10
#### `analyze_directories`

**Optional** A space seperated list of folders that should be analyzed.
**Optional** A space separated list of folders that should be analyzed.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

caught some typos❗

@felangel felangel force-pushed the feat/spell-check branch 4 times, most recently from 06ced12 to 94a8997 Compare October 11, 2022 04:26
"dictionaries": ["vgv_allowed", "vgv_forbidden"],
"dictionaryDefinitions": [
{
"name": "vgv_allowed",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"name": "vgv_allowed",
"name": "very_allowed_words",

Joking, but would be cool :P

"description": "Allowed VGV Spellings"
},
{
"name": "vgv_forbidden",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"name": "vgv_forbidden",
"name": "very_forbidden_words",

Joking, but would be cool :P

@felangel felangel merged commit 9c12bb5 into main Oct 11, 2022
@felangel felangel deleted the feat/spell-check branch October 11, 2022 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature A new feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants