Skip to content

Conversation

@pdesoyres-cc
Copy link
Contributor

@pdesoyres-cc pdesoyres-cc commented Oct 21, 2025

Fixes #177

What this PR do?

It uses @clevercloud/eslint-config

Some configs are overridden and are waiting for one PR to be merged:

@pdesoyres-cc pdesoyres-cc self-assigned this Oct 21, 2025
@pdesoyres-cc pdesoyres-cc force-pushed the use-eslint-config-from-js-tooling branch from bee0ae2 to 72a92e6 Compare October 21, 2025 15:25
Copy link
Contributor

@florian-sanders-cc florian-sanders-cc left a comment

Choose a reason for hiding this comment

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

Thanks for handling this @pdesoyres-cc (+ the associated PRs in the tooling project ❤️)

I only have a small question, not directly related to your changes.
Apart from that, LGTM but @Galimede's opinion is more important than mine since he's the one who handled the initial config setup in this project 👍

Copy link
Member

@Galimede Galimede left a comment

Choose a reason for hiding this comment

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

Hey @pdesoyres-cc, thanks for handling this, LGTM! 💪

@pdesoyres-cc pdesoyres-cc force-pushed the use-eslint-config-from-js-tooling branch from 72a92e6 to c87281c Compare November 13, 2025 08:39
@pdesoyres-cc pdesoyres-cc force-pushed the use-eslint-config-from-js-tooling branch from c87281c to 358ef43 Compare November 13, 2025 09:31
@pdesoyres-cc pdesoyres-cc merged commit e39682d into master Nov 13, 2025
2 checks passed
@pdesoyres-cc pdesoyres-cc deleted the use-eslint-config-from-js-tooling branch November 13, 2025 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

use common eslint config

4 participants