chore: disable cra eslint (#24953)

## Description
Disable CRA built-in ESLint checks since we have our own config and a
separate step for this

#### Type of change
- Chore (housekeeping or task changes that don't impact user perception)

## Testing
>
#### How Has This Been Tested?
> Please describe the tests that you ran to verify your changes. Also
list any relevant details for your test configuration.
> Delete anything that is not relevant
- [x] Manual
- [ ] Jest
- [ ] Cypress

## Checklist:
#### Dev activity
- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] PR is being merged under a feature flag

Co-authored-by: Valera Melnikov <melnikov.vv@greendatasoft.ru>
This commit is contained in:
Valera Melnikov 2023-06-30 12:45:07 +03:00 committed by GitHub
parent 35241621dd
commit d7505e77c9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 2 deletions

View File

@ -53,8 +53,8 @@
"error",
{ "caseSensitive": false }
],
"no-console": "warn",
"no-debugger": "warn",
"no-console": "error",
"no-debugger": "error",
"@typescript-eslint/no-restricted-imports": [
"error",
{

View File

@ -18,6 +18,8 @@ fi
# build cra app
export REACT_APP_SENTRY_RELEASE=$GIT_SHA
export REACT_APP_CLIENT_LOG_LEVEL=ERROR
# Disable CRA built-in ESLint checks since we have our own config and a separate step for this
export DISABLE_ESLINT_PLUGIN=true
craco --max-old-space-size=7168 build --config craco.build.config.js
if [ "$GITHUB_REPOSITORY" == "appsmithorg/appsmith-ee" ]; then