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

[INFRA] Configure a linter #11

Closed
tbouffard opened this issue Jul 18, 2023 · 0 comments · Fixed by #84
Closed

[INFRA] Configure a linter #11

tbouffard opened this issue Jul 18, 2023 · 0 comments · Fixed by #84
Assignees
Labels
chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...)
Milestone

Comments

@tbouffard
Copy link
Member

tbouffard commented Jul 18, 2023

We should apply a configuration close or identical to the one used in https://github.com/process-analytics/bpmn-visualization-js
Be aware of process-analytics/bpmn-visualization-js#2742 and process-analytics/bpmn-visualization-js#2771

As a first implementation, we will duplicate the eslint configuration from the bpmn-visualization repository, then we will use a shared configuration when it is available: #27.
Using the same eslint config as this repository will ease the move of code between the 2 repositories, for instance when

  • extracting demo code from the lib to the extension
  • graduating experimental features to the lib
@tbouffard tbouffard added the chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...) label Jul 18, 2023
@tbouffard tbouffard added this to the 0.2.0 milestone Jul 19, 2023
@tbouffard tbouffard modified the milestones: 0.2.0, 0.3..0 Jul 24, 2023
@tbouffard tbouffard modified the milestones: 0.3.0, 0.4.0 Aug 21, 2023
@tbouffard tbouffard self-assigned this Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant