maoertel / hoconvert Goto Github PK

Hi 👋

First of all, thanks for this great tool. Unfortunately, it has one major flaw, because it doesn't work for files with includes. That is 99% of the files I'd like to use it with, because we usually include files with separate values for development and production, in order to not duplicate the entire config.

The hoconvert returns Error: Hocon(IncludeNotAllowedFromStr) for such files, because the underlying HoconLoader doesn't support includes when reading from string. It is however supported when loading from file. It should not be very difficult to modify the code to pass a file path to the HoconLoader.load_file if the user supplies the path on command line.

For HOCON passed in through stdin the behaviour can stay the same. It doesn't make much sense for the most common use cases anyway.

The tool JC https://github.com/kellyjonbrazil/jc has been for me an epicenter for converting things into json.

To the point that ansible integrated it into their pipeline.

Would you want to connect to that maintainer and pull this code into his project?

I just want a general tool to rule them all

Hi,
Great project!

Would you be open to publishing a docker image for hoconvert?