Skip to content

Commit

Permalink
Update heck to 0.4.0
Browse files Browse the repository at this point in the history
  • Loading branch information
messense authored and emilio committed Mar 25, 2022
1 parent 88d4b99 commit 293fcc5
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 15 deletions.
13 changes: 2 additions & 11 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion Cargo.toml
Expand Up @@ -30,7 +30,7 @@ tempfile = "3.0"
toml = "0.5"
proc-macro2 = "1"
quote = "1"
heck = "0.3"
heck = "0.4"

[dependencies.syn]
version = "1.0.3"
Expand Down
6 changes: 3 additions & 3 deletions src/bindgen/rename.rs
Expand Up @@ -69,11 +69,11 @@ impl RenameRule {

Cow::Owned(match self {
RenameRule::None => return Cow::Borrowed(text),
RenameRule::GeckoCase => context.to_str().to_owned() + &text.to_camel_case(),
RenameRule::GeckoCase => context.to_str().to_owned() + &text.to_upper_camel_case(),
RenameRule::LowerCase => text.to_lowercase(),
RenameRule::UpperCase => text.to_uppercase(),
RenameRule::PascalCase => text.to_camel_case(),
RenameRule::CamelCase => text.to_mixed_case(),
RenameRule::PascalCase => text.to_pascal_case(),
RenameRule::CamelCase => text.to_upper_camel_case(),

This comment has been minimized.

Copy link
@jrose-signal

jrose-signal Apr 4, 2022

Contributor

Was this intentional? How do I get lowerCamelCase now?

This comment has been minimized.

Copy link
@emilio

emilio Apr 4, 2022

Collaborator

If this changed behavior then it's not intentional. Feel free to send a fix + a test? :)

This comment has been minimized.

Copy link
@jrose-signal

jrose-signal Apr 5, 2022

Contributor

#750!

RenameRule::SnakeCase => text.to_snake_case(),
RenameRule::ScreamingSnakeCase => text.to_shouty_snake_case(),
RenameRule::QualifiedScreamingSnakeCase => {
Expand Down

0 comments on commit 293fcc5

Please sign in to comment.