Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

Use TOML as default storage #2330

Closed
2 tasks
markus2330 opened this issue Jan 23, 2019 · 3 comments
Closed
2 tasks

Use TOML as default storage #2330

markus2330 opened this issue Jan 23, 2019 · 3 comments

Comments

@markus2330
Copy link
Contributor

markus2330 commented Jan 23, 2019

The current default format "dump" is not suitable to be edited by hand (e.g. by using kdb editor), which is a usability problem. To improve the situation, YAML can be used. The new storage plugin should fulfill following properties (ordered by priority):

  1. it should round-trip (if a KeySet is serialized and then parsed again, we end up with an identical internal representation)
  2. drop-in replacement as default storage plugin even if user still has some dump files lying around (something like dini is needed)
  3. user efforts in manually crafting the file are not lost (comments, ordering, ...)
  4. source code is maintainable (grammar-based,...) and modular (parsing, generation, ... separated)
  5. good error messages for typical mistakes
  6. good documentation of the format (and grammar for formal definition)
  7. line number information is kept linenumbers for validation #736

Tasks:

@markus2330 markus2330 added this to the 1.0.0 milestone Jan 23, 2019
@markus2330 markus2330 changed the title Use YAML as default storage Use TOML as default storage Nov 14, 2019
@markus2330 markus2330 assigned bauhaus93 and unassigned sanssecours Nov 14, 2019
@markus2330
Copy link
Contributor Author

@bauhaus93 What is the current status?

@stale
Copy link

stale bot commented Aug 7, 2021

I mark this issue stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping the issue by writing a message here or create a new issue with the remainder of this issue.
Thank you for your contributions 💖

@stale stale bot added the stale label Aug 7, 2021
@stale
Copy link

stale bot commented Aug 21, 2021

I closed this issue now because it has been inactive for more than one year. If I closed it by mistake, please do not hesitate to reopen it or create a new issue with the remainder of this issue.
Thank you for your contributions 💖

@stale stale bot closed this as completed Aug 21, 2021
This was referenced Sep 27, 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

3 participants