Skip to content

fix: attribute get_mut supports with_prefix#108

Merged
jbaublitz merged 1 commit intojbaublitz:mainfrom
jason-hchsieh:fix-get-mut-with-prefix
Feb 26, 2025
Merged

fix: attribute get_mut supports with_prefix#108
jbaublitz merged 1 commit intojbaublitz:mainfrom
jason-hchsieh:fix-get-mut-with-prefix

Conversation

@jason-hchsieh
Copy link
Copy Markdown
Contributor

Fixed #107

@jbaublitz
Copy link
Copy Markdown
Owner

Hi @jason-hchsieh, thanks for the contribution. For starters, if you could squash the format commit now that would be great. I'm going to take a look at the code now.

Copy link
Copy Markdown
Owner

@jbaublitz jbaublitz left a comment

Choose a reason for hiding this comment

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

The changes are pretty straightforward, this looks good. Thank you for adding to the tests!

@jbaublitz jbaublitz added the bug label Feb 25, 2025
@jbaublitz jbaublitz added this to the v0.1.5 milestone Feb 25, 2025
@jason-hchsieh jason-hchsieh force-pushed the fix-get-mut-with-prefix branch from 992cc96 to 0b52e94 Compare February 26, 2025 05:06
@jason-hchsieh
Copy link
Copy Markdown
Contributor Author

Hi @jason-hchsieh, thanks for the contribution. For starters, if you could squash the format commit now that would be great. I'm going to take a look at the code now.

I've done the commit squash. Let me know if there's anything else needed. Thanks!

@jbaublitz jbaublitz merged commit e0a03da into jbaublitz:main Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The get_mut = "with_prefix" configured in the struct field does not have the get_ prefix.

2 participants