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

Serde Support #40

Open
Michael-F-Bryan opened this issue Dec 22, 2017 · 7 comments
Open

Serde Support #40

Michael-F-Bryan opened this issue Dec 22, 2017 · 7 comments

Comments

@Michael-F-Bryan
Copy link

What are your thoughts on adding serde support so you can serialize to/deserialize from an INI file? Along the same lines as serde_json, toml, or serde_xml.

@zonyitoo
Copy link
Owner

Well, INI file format is a flat format, which is a pure and simple key-value structure. So I don't think it is necessary to be used with serde. Ini is just a two level HashMap.

@hellow554
Copy link

I think it's nice to have! It's much cleaner and simpler to let serde parse the values into the correct types, instead of doing that by hand.

So I would think this is worth supporting.

@zonyitoo
Copy link
Owner

zonyitoo commented Jun 3, 2019

PR is welcomed. Because I don't have time recently :(

@kirawi
Copy link

kirawi commented Sep 11, 2021

I'll try to do this, though I can't guarantee I'll have the time either.

@Zerowalker
Copy link

Zerowalker commented Nov 20, 2021

I would also like this. I am trying my hands at it but proc_macro stuff is not something i have worked with before.
made some progress, i can deserialize to basic String and Option, but not sections.

My goal is to support sections and allow duplicate ones to make a Vec<...> out of it.

@kirawi have you made any progress?

Other than that, great crate, it's the only one that supported the ini files i am trying to parse that i could find,
and quite straight forward:)

EDIT: Forgot to mention that it's not using serde right now, just doing it "manually" so to speak.

@kirawi
Copy link

kirawi commented Nov 20, 2021

Honestly, I'll admit I completely forgot 😅 you can go ahead.

@Zerowalker
Copy link

Haha xd
Well will do my best, but don't expect much;)

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

No branches or pull requests

5 participants