Fix conflicts in eslint
package versions (#2219)
#2220
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #2219
Changes:
eslint-config-airbnb
to the latest version.eslint
from7.31.0
to7.32.0
.0
) rules which I think that we don't want to keep.1
) rules which I think that we should keep.else
branch inclient/modules/IDE/pages/MobileIDEView.jsx
(violatedno-dupe-else-if
)FileActions
inclient/modules/IDE/components/FileNode.jsx
(violatedno-import-assign
)export { ConnectedFileNode as default };
toexport default ConnectedFileNode;
inclient/modules/IDE/components/FileNode.jsx
(violatedno-restricted-exports
)client/modules/User/actions.js
so that the inner function of thePromise
doesn't return anything (violatedno-promise-executor-return
)ChangedI had to roll this back due to issues with Jest.if (module.hot)
toif (import.meta.webpackHot)
inclient/store.js
as recommended by webpack for strict ESM compliance. (violatedimport/no-import-module-exports
)RedirectToUser
. (violatedreact/prop-types
)I have verified that this pull request:
npm run lint
)npm run test
)develop
branch.Fixes #123
There are no linting errors but there are 64 warnings.