-
Notifications
You must be signed in to change notification settings - Fork 151
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Custom prefix/case/suffix for identifiers #794
Conversation
ce76aad
to
d1f2458
Compare
I like it :) |
Could we merge this as it is? |
serde(rename_all = "lowercase") | ||
)] | ||
#[derive(Clone, Debug, PartialEq, Eq, Default)] | ||
pub enum Case { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make this non-exhaustive
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
/ci diff pr |
is the diff ok? @@ -87431,13 +87431,13 @@ pub struct Peripherals {
pub OTG_FS_HOST: OTG_FS_HOST,
///OTG_FS_PWRCLK
pub OTG_FS_PWRCLK: OTG_FS_PWRCLK,
- ///ETHERNET_MMC
+ ///Ethernet_MMC
pub ETHERNET_MMC: ETHERNET_MMC,
- ///ETHERNET_MAC
+ ///Ethernet_MAC
pub ETHERNET_MAC: ETHERNET_MAC,
- ///ETHERNET_PTP
+ ///Ethernet_PTP
pub ETHERNET_PTP: ETHERNET_PTP,
- ///ETHERNET_DMA
+ ///Ethernet_DMA
pub ETHERNET_DMA: ETHERNET_DMA,
///SCB_ACTRL
pub SCB_ACTRL: SCB_ACTRL, |
Yes. I think it is better do not change case of peripheral name in documentation. |
This PR doesn't add new cli options (yet?). But you can pass options by config:
cc @Emilgardis what do you think?
This should not be a breaking change.