pretty-format-toml removes trailing comments in list

[l0b0]

When reformatting a list setting like

load-plugins = [
  "pylint.extensions.bad_builtin",
  # "pylint.extensions.docparams",
  "pylint.extensions.set_membership"
  # "pylint.extensions.typing"
]

, the first comment is retained, but the second one is removed. Both comments should be retained. Not sure whether this is an issue with TomlSort or this repo.

[maresb]

This looks like a duplicate of #161

[maresb]

I feel like this TOML pre-commit hook needs a big red warning that it's unsafe since it will unceremoniously delete comments.

[Avasam]

I find it'll do so even outside lists. And I have no workaround, making this hook pretty unusable.
It also removes newlines between blocks of comments

[corneliusroemer]

Yes, TOML needs to be fixed or marked unsafe as it removes comments - don't use!

[maresb]

@corneliusroemer I don't see a corresponding PR for the commit linked above. Is one on the way, or did I miss it? Thanks for following through with that!

[corneliusroemer]

@maresb That is for reminding me! I must have forgotten to make a PR. Done now! #242

[maresb]

By the way, if people need a safe TOML pre-commit hook, this just became available a few days ago and it seems to work very well. (The previous version used a cumbersome Docker contaier.):

https://github.com/ComPWA/mirrors-taplo