Skip to content

Conversation

@williamstar
Copy link

No description provided.

@davidhewitt
Copy link
Member

Thanks, I'll try to take a look at this soon! My review backlog is currently a few days long, so I'm afraid it might be early next week when I look at this.

@williamstar
Copy link
Author

👌🏻

@davidhewitt
Copy link
Member

Thank you for this implementation, which showed me how we can support this.

When reviewing I decided I prefer a slightly different way for the user to configure kebab-case, by using the target dictionary to indicate kebab-case directly. I think that's a slightly more intuitive approach than having the script dictionary chain the mapping as proposed here. Opened it as #205

@williamstar
Copy link
Author

The implementation looks great.

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.

2 participants