Skip to content

👷 Disable Eslint in Codeclimate#3104

Closed
josemigallas wants to merge 3 commits into3scale:masterfrom
josemigallas:disable_eslint_codeclimate
Closed

👷 Disable Eslint in Codeclimate#3104
josemigallas wants to merge 3 commits into3scale:masterfrom
josemigallas:disable_eslint_codeclimate

Conversation

@josemigallas
Copy link
Contributor

We already run the same command in Circleci.

@josemigallas josemigallas requested a review from a team November 10, 2022 07:36
@josemigallas josemigallas self-assigned this Nov 10, 2022
jlledom
jlledom previously approved these changes Nov 10, 2022
nidhi-soni1104
nidhi-soni1104 previously approved these changes Nov 10, 2022
Copy link
Contributor

@nidhi-soni1104 nidhi-soni1104 left a comment

Choose a reason for hiding this comment

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

LGTM

@nidhi-soni1104
Copy link
Contributor

juts fix code climate

@akostadinov
Copy link
Contributor

codeclimate issue is rather strange

@josemigallas josemigallas dismissed stale reviews from nidhi-soni1104 and jlledom via f399d64 November 10, 2022 11:26
@josemigallas josemigallas force-pushed the disable_eslint_codeclimate branch from 9ad1628 to f399d64 Compare November 10, 2022 11:26
@josemigallas
Copy link
Contributor Author

josemigallas commented Nov 10, 2022

codeclimate issue is rather strange

I think my fork was missing a rebase.

EDIT: it wasn't that. It seems the Eslint check is still running even thought the plugin is disabled. I've contacted codeclimate...

@josemigallas josemigallas force-pushed the disable_eslint_codeclimate branch from 6228ae8 to ee9f196 Compare November 10, 2022 15:07
@akostadinov
Copy link
Contributor

maybe they don't read config from fork branches - only from main repo. Like circleci

@josemigallas
Copy link
Contributor Author

josemigallas commented Nov 10, 2022

maybe they don't read config from fork branches - only from main repo. Like circleci

I created a branch in the original repo (#3106) and it's still not working 😕 The only explanation possible is that plugin cannot be disabled.
We can also try merging and see if it has any effect from master.

@josemigallas
Copy link
Contributor Author

Already merged in #3106

@josemigallas josemigallas deleted the disable_eslint_codeclimate branch November 16, 2022 15:21
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.

4 participants