Skip to content

fix: typo#1741

Merged
ellie merged 1 commit into
atuinsh:mainfrom
c-git:patch-1
Feb 20, 2024
Merged

fix: typo#1741
ellie merged 1 commit into
atuinsh:mainfrom
c-git:patch-1

Conversation

@c-git

@c-git c-git commented Feb 20, 2024

Copy link
Copy Markdown
Contributor

Fixes a small typo

Checks

  • I am happy for maintainers to push small adjustments to this PR, to speed up the review cycle
  • I have checked that there are no existing pull requests for the same thing

@c-git

c-git commented Feb 20, 2024

Copy link
Copy Markdown
Contributor Author

I just realized it might just be the the setting is actually spelled that way and isn't wrong. I didn't actually test just noticed the underline in vscode.

@akinomyoga

akinomyoga commented Feb 20, 2024

Copy link
Copy Markdown
Contributor

It's my fault. edit: It should be just a typo unless I made exactly the same typo in the actual code, which is unlikely. edit2: I confirmed:

$ git grep underilne
atuin-client/config.toml:## "{blink,steady}-{block,underilne,bar}".
$ git grep underline
atuin-client/src/settings.rs:    #[serde(rename = "blink-underline")]
atuin-client/src/settings.rs:    #[serde(rename = "steady-underline")]

@ellie

ellie commented Feb 20, 2024

Copy link
Copy Markdown
Member

Thank you! Speedy review there @akinomyoga, thank you 🥳

Seeing as this is your first time contributing, if you would like a holographic contributors-only Atuin sticker, then please fill out this form!

@ellie ellie merged commit 409456f into atuinsh:main Feb 20, 2024
@c-git c-git deleted the patch-1 branch February 20, 2024 14:31
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.

3 participants