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

chore(eslint): add eslint-plugin-import dependency #2842

Merged
merged 27 commits into from
Sep 6, 2023

Conversation

csouchet
Copy link
Member

@csouchet csouchet commented Sep 1, 2023

This pull request addresses linting and import-related issues. Major changes include:

Covers #2742
Closes #2078

@csouchet csouchet added dependencies Pull requests that update a dependency (dev or runtime) chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...) labels Sep 1, 2023
@github-actions
Copy link

github-actions bot commented Sep 1, 2023

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

🤖 By surge-preview

@github-actions
Copy link

github-actions bot commented Sep 1, 2023

🎊 PR Preview d066927 has been successfully built and deployed to https://process-analytics-bpmn-visualization-js-doc_preview-pr-2842.surge.sh

🕐 Build time: 0.02s

🤖 By surge-preview

@csouchet csouchet marked this pull request as ready for review September 4, 2023 07:19
@csouchet csouchet requested a review from tbouffard September 4, 2023 07:19
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, this should simplify the review of the future PR. Good job, this plugin does the job well!

The review was tedious because I checked all the modified files, but it was worth it because I detected a few minor problems. 😄

.eslintrc.cjs Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
rollup.config.mjs Outdated Show resolved Hide resolved
scripts/generate-types-for-not-supported-ts-versions.mjs Outdated Show resolved Hide resolved
@@ -26,7 +26,6 @@ export type { Navigation } from './component/navigation';
export * from './model/bpmn/internal';

// not part of the public API but needed for the custom theme examples
export { IconPainter } from './component/mxgraph/shape/render/icon-painter';
Copy link
Member

Choose a reason for hiding this comment

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

praise: nice catch 😸

@csouchet csouchet requested a review from tbouffard September 4, 2023 15:02
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
Can you please answer to the open questions and integrate the proposed changes? As usual, we can quickly talk about them to converge and be able to merge 😸.

test/typescript-support/src/index.ts Outdated Show resolved Hide resolved
scripts/generate-types-for-not-supported-ts-versions.mjs Outdated Show resolved Hide resolved
rollup.config.mjs Outdated Show resolved Hide resolved
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@csouchet csouchet merged commit 735e5a8 into master Sep 6, 2023
@csouchet csouchet deleted the 2742-add_eslint-plugin-import branch September 6, 2023 08:52
@tbouffard tbouffard removed the dependencies Pull requests that update a dependency (dev or runtime) label Sep 7, 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 this pull request may close these issues.

[INFRA] Ensure consistent imports order in code and tests
2 participants