Skip to content

fix(lint): ensure dictionary.txt contains inputs#33

Merged
JonZeolla merged 2 commits intomainfrom
ensure-spaces-are-new-tokens
Jul 9, 2025
Merged

fix(lint): ensure dictionary.txt contains inputs#33
JonZeolla merged 2 commits intomainfrom
ensure-spaces-are-new-tokens

Conversation

@JonZeolla
Copy link
Copy Markdown
Member

Contributor Comments

This ensures that the generated dictionary.txt has all of the various inputs, even if they have spaces they should be separate entries (like "Jon Zeolla")

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

Copy link
Copy Markdown

@ai-coding-guardrails ai-coding-guardrails Bot left a comment

Choose a reason for hiding this comment

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

Nice work! 😎

I didn't find anything of concern

Reviewed with 🤟 by Zenable

@JonZeolla JonZeolla enabled auto-merge (squash) July 9, 2025 23:45
@JonZeolla JonZeolla merged commit 154541b into main Jul 9, 2025
8 checks passed
@JonZeolla JonZeolla deleted the ensure-spaces-are-new-tokens branch July 9, 2025 23:51
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

Successfully merging this pull request may close these issues.

1 participant