Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP feat: add command line utility #120

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

WIP feat: add command line utility #120

wants to merge 1 commit into from

Conversation

philippfromme
Copy link
Collaborator

Makes using the linter from the command line (e.g., linting test/spec/*.bpmn) simple.

@bpmn-io-tasks bpmn-io-tasks bot added the in progress Currently worked on label Oct 16, 2024
@nikku
Copy link
Member

nikku commented Oct 17, 2024

I'm not sure I understand the use-case for using this library as a CLI tool for linting.

What keeps folks from just using stock bpmnlint + the compat rules?

@philippfromme
Copy link
Collaborator Author

What keeps folks from just using stock bpmnlint + the compat rules?

You can use this without any configuration which is true for both browser and CLI usage.

@nikku
Copy link
Member

nikku commented Oct 17, 2024

But also it is less pluggable, i.e. does not play nicely with custom lint rules, right?

@barmac
Copy link
Contributor

barmac commented Oct 17, 2024

From the command-line utility perspective, the main advantage of @camunda/linting that I can see is the ability to adjust the configuration dynamically based on the diagram target execution version. However, I am not sure if this serves governance well. I imagine that as a diagrams owner, I know where I deploy to, and can set the expected target platform (explicit over implicit). Is there anything I missed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in progress Currently worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants