Skip to content

Update for latest typos#139

Merged
zuiderkwast merged 1 commit into
valkey-io:unstablefrom
jayvdb:update-typos
Apr 2, 2024
Merged

Update for latest typos#139
zuiderkwast merged 1 commit into
valkey-io:unstablefrom
jayvdb:update-typos

Conversation

@jayvdb

@jayvdb jayvdb commented Apr 2, 2024

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: john vandenberg <jayvdb@gmail.com>
@jayvdb

jayvdb commented Apr 2, 2024

Copy link
Copy Markdown
Contributor Author

@zuiderkwast this ignores most of the latest batch. Many of these could be updated in the source to use more clear names that would avoid the extra whitelisting, but I've only fixed the clear spelling mistakes.

@zuiderkwast zuiderkwast left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Awesome

@zuiderkwast

Copy link
Copy Markdown
Contributor

Can we pin it to a specific version, so things don't start to break whenever there is a new version?

@zuiderkwast zuiderkwast merged commit 83507d7 into valkey-io:unstable Apr 2, 2024
@jayvdb jayvdb deleted the update-typos branch April 2, 2024 12:52
@jayvdb

jayvdb commented Apr 2, 2024

Copy link
Copy Markdown
Contributor Author

Yes, you can change https://github.com/valkey-io/valkey/blob/unstable/.github/workflows/spell-check.yml#L22 to be a specific version of the action, like https://github.com/taiki-e/install-action/releases/tag/v2.32.2 , and it will be more stable

PatrickJS pushed a commit to PatrickJS/placeholderkv that referenced this pull request Apr 24, 2024
Signed-off-by: john vandenberg <jayvdb@gmail.com>
Co-authored-by: john vandenberg <jayvdb@192-168-1-101.tpgi.com.au>
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.

2 participants