bpmn-io / bpmn-to-image

Convert BPMN 2.0 diagrams to PDF documents or PNG files.
62 stars 38 forks source link

[BREAKING] Rework CLI path format #16

Closed Morozzzko closed 3 years ago

Morozzzko commented 4 years ago

Relates to #15

This is a first step towards introduction of stdin and stdout as input/output streams.

Breaking changes:

The format is now bpmn-to-image --destination=<outputConfig> <diagramFile>. As a consequence, you can only convert one source per run.

This enables us to move forward to using stdin as input source

CLAassistant commented 4 years ago

CLA assistant check
All committers have signed the CLA.

MaxTru commented 4 years ago

Thanks for the PR!

stdin certainly allows for more convenient / flexible usage of the cli.

As a consequence, you can only convert one source per run.

From my understanding, also in the previous format only one bpmn diagram would be accepted as source. So from this perspective, this PR would not reduce the existing functionality.

Could you please also include the changed format in the example/documentation in https://github.com/bpmn-io/bpmn-to-image/blob/master/README.md

Could you please adjust the Contribution according to contribution guidelines? In particular: