Skip to content
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 dict delimiter #277

Merged
merged 1 commit into from
Nov 8, 2023
Merged

Conversation

gregoiredx
Copy link
Contributor

@gregoiredx gregoiredx commented Oct 21, 2022

Allow specifying the delimiter for dict as it is already the case for list.

@gregoiredx gregoiredx marked this pull request as ready for review October 21, 2022 12:19
@gregoiredx gregoiredx mentioned this pull request Oct 21, 2022
@gregoiredx
Copy link
Contributor Author

gregoiredx commented Oct 26, 2022

@sloria , WDYT about this? Does it makes sense to you?

@gregoiredx
Copy link
Contributor Author

@sloria , a little up on this one. Can you merge this? Or should I close the PR?

@gregoiredx
Copy link
Contributor Author

@sloria , it's been a year 😄
Should we merge this PR or just close it?

@sloria
Copy link
Owner

sloria commented Nov 8, 2023

Sorry for the delay here. This looks good. I need to update CI to use github actions but haven't had the time.

I'll go ahead and merge this but won't be able to release until CI is fixed.

@sloria sloria merged commit f436546 into sloria:master Nov 8, 2023
@gregoiredx
Copy link
Contributor Author

cool thanks

@sloria
Copy link
Owner

sloria commented Dec 15, 2023

this is now released in 10.0.0

@gregoiredx
Copy link
Contributor Author

Yeah 🎉 Thanks

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