Skip to content

Conversation

csouchet
Copy link
Contributor

No description provided.

@csouchet csouchet added the chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...) label Feb 24, 2025
Copy link

github-actions bot commented Feb 24, 2025

♻️ PR Preview c1d054b has been successfully destroyed since this PR has been closed.

🤖 By surge-preview

@csouchet csouchet marked this pull request as ready for review February 24, 2025 10:26
@csouchet csouchet requested a review from tbouffard February 24, 2025 10:26
@csouchet csouchet force-pushed the chore/convert_eslint_config_to_flat branch from 079d073 to 55ed97c Compare February 26, 2025 09:02
@csouchet csouchet force-pushed the chore/convert_eslint_config_to_flat branch from 55ed97c to 4ad406d Compare March 10, 2025 09:22
Copy link

Copy link
Member

@tbouffard tbouffard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some additional linting rules which are added whereas we only want to switch the configuration.
Review the changes and if we want to apply the new rules, apply them to the main branch first.

this.model = bpmnVisualization.graph.model;
}

/* eslint-disable jest/no-standalone-expect -- util code, including expect calls */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question: why is this removed?

jest: ensure recommended and style rules are enabled + apply jsdom config correctly

test files:
  - use right glob
  - centralize config

remove extra unicorn configuration that didn't exist in the former configuration
The extra configuration came from bpmn-visualization and had been copy/paste here by mistake
# Conflicts:
#	package-lock.json
#	packages/addons/package.json
Copy link

Copy link
Member

@tbouffard tbouffard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, all plugins have been tested individually: they detect and can auto-fix errors/warnings.

@tbouffard tbouffard merged commit 6209c68 into main Jun 20, 2025
5 checks passed
@tbouffard tbouffard deleted the chore/convert_eslint_config_to_flat branch June 20, 2025 07:01
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 this pull request may close these issues.

2 participants