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

Adding serde (de)serialization as optional feature #30

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mdsherry
Copy link
Owner

@mdsherry mdsherry commented Jun 8, 2022

Plus a couple of lint fixes

@SebRut
Copy link

SebRut commented Sep 13, 2022

Anything preventing this to be merged?

@mdsherry
Copy link
Owner Author

@SebRut : If you need this, could you explain the use case to me? The code in the PR works, but I don't know that it's useful, and that it does what people would actually want it to do.

@SebRut
Copy link

SebRut commented Sep 16, 2022

I was thinking about using this to deserialize time intervals for schedules from config files with serde. I think right now the workaround would be to have fields like "intervalMinutes" and convert these manually later.

@imstevenxyz
Copy link

I have the same use case as @SebRut: intervals deserialized from a config file.

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.

3 participants