Skip to content

fix: (cli) format plugin string to snake case, closes #8017#8149

Merged
lucasfernog merged 4 commits intotauri-apps:devfrom
vasfvitor:cli-snake-case
Nov 8, 2023
Merged

fix: (cli) format plugin string to snake case, closes #8017#8149
lucasfernog merged 4 commits intotauri-apps:devfrom
vasfvitor:cli-snake-case

Conversation

@vasfvitor
Copy link
Copy Markdown
Contributor

@vasfvitor vasfvitor commented Nov 1, 2023

@vasfvitor vasfvitor requested a review from a team as a code owner November 1, 2023 02:21
Copy link
Copy Markdown
Member

@amrbashir amrbashir left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution, Can you also add a change file in .changes directory?

@vasfvitor
Copy link
Copy Markdown
Contributor Author

vasfvitor commented Nov 7, 2023

Done. Anything else?

Should I include @tauri-apps/cli too?

@lucasfernog lucasfernog requested a review from amrbashir November 7, 2023 16:24
@lucasfernog
Copy link
Copy Markdown
Member

We actually use the heck crate to convert cases but the replace here is ok.

@vasfvitor
Copy link
Copy Markdown
Contributor Author

We actually use the heck crate to convert cases but the replace here is ok.

Thanks for mentioning it. I should've stick to my guns. I did see to_snake_case() around but didn't want to import it.

@lucasfernog lucasfernog merged commit 88dac86 into tauri-apps:dev Nov 8, 2023
@vasfvitor vasfvitor deleted the cli-snake-case branch March 3, 2024 16:57
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