metafacture / metafacture-documentation

The central place for documentation about metafacture
http://metafacture.github.io/metafacture-documentation/
Apache License 2.0
3 stars 3 forks source link

Delete prettyPrinting option for encode-yaml in documentation #19

Closed TobiasNx closed 1 month ago

TobiasNx commented 2 years ago

This option is not supported. So I would just delete it.

blackwinter commented 2 years ago

Background: Jackson's YAMLGenerator ignores any PrettyPrinter (and in version 3 it's completely gone). There is a feature request (FasterXML/jackson-dataformats-text#43) that might help in this regard, but it hasn't been implemented yet.

dr0i commented 2 years ago

The option is supported , as it always does pretty printing. It's just superfluous.

blackwinter commented 2 years ago

The option is supported , as it always does pretty printing.

But you can't disable pretty printing as the documentation suggests.

dr0i commented 2 years ago

Ah, I see. Yeah, that should be noted somehow.

dr0i commented 2 years ago

Closed in favour of https://github.com/metafacture/metafacture-core/pull/447.

dr0i commented 2 years ago

Sorry, not the issue should be closed, but another PR.

dr0i commented 2 years ago

Updated https://github.com/metafacture/metafacture-documentation/blob/master/flux-commands.md#encode-yaml .

TobiasNx commented 1 month ago

Resolved.