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

Switch to a more up to date TOML library #2089

Closed
spf13 opened this issue Apr 17, 2016 · 8 comments
Closed

Switch to a more up to date TOML library #2089

spf13 opened this issue Apr 17, 2016 · 8 comments
Assignees
Milestone

Comments

@spf13
Copy link
Contributor

spf13 commented Apr 17, 2016

https://github.com/pelletier/go-toml seems like a good candidate.

@spf13 spf13 self-assigned this Apr 17, 2016
@pelletier
Copy link

Maintainer of https://github.com/pelletier/go-toml here. Let me know if there is anything I can do to help!

@mohae
Copy link
Contributor

mohae commented May 5, 2016

It appears that https://github.com/pelletier/go-toml only does parsing. Encoding would be needed too; unless I'm missing something.

@bep
Copy link
Member

bep commented May 6, 2016

Not sure what you refer to by encoding, but in the latest revision, you can get a "map of plain Go objects", which is what we need.

If it is more "up to date" (whatever that means; I'm guessing supporting more of the TOML spec and not more up to date in tech fashion, although there have been some fishy stuff in what we use now; an os.Exit on error being one I remember).

@bep
Copy link
Member

bep commented May 6, 2016

Or: Do we ever write TOML?

@mohae
Copy link
Contributor

mohae commented May 6, 2016

@pelletier
Copy link

@mohae go-toml does serialize back to a string, see https://godoc.org/github.com/pelletier/go-toml#TomlTree.ToString

@mohae
Copy link
Contributor

mohae commented May 8, 2016

@pelletier thanks for the info! I missed that.

@bep bep added the Enhancement label Aug 5, 2016
@bep bep added this to the v0.17 milestone Aug 5, 2016
@bep bep closed this as completed in 20c4311 Aug 20, 2016
tychoish pushed a commit to tychoish/hugo that referenced this issue Aug 13, 2017
@github-actions
Copy link

github-actions bot commented Apr 4, 2022

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants