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

A few updates post eslint #4425

Merged
merged 4 commits into from
Dec 9, 2019
Merged

A few updates post eslint #4425

merged 4 commits into from
Dec 9, 2019

Conversation

gabrieldutra
Copy link
Member

What type of PR is this? (check all applicable)

  • Other

Description

@arikfr here are a few updates I had in mind (was quicker to share in a PR than in comments). The ones @kravets-levko mentioned I'm not so sure about, so will confirm them first.

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

@arikfr arikfr requested a review from kravets-levko December 8, 2019 08:12
@@ -86,13 +86,13 @@ function EditParameterSettingsDialog(props) {

// fetch query by id
useEffect(() => {
const { queryId } = props.parameter;
const queryId = props.parameter.queryId;
Copy link
Member

Choose a reason for hiding this comment

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

Is this change needed for eslint to detect the correct value for the dependencies array?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, using it like this it automatically corrects to [props.parameter.queryId]

Copy link
Member

@arikfr arikfr left a comment

Choose a reason for hiding this comment

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

Merge at will.

@gabrieldutra
Copy link
Member Author

@kravets-levko ath else you think it's important here?

@gabrieldutra gabrieldutra merged commit 5936249 into eslint Dec 9, 2019
@gabrieldutra gabrieldutra deleted the eslint-updates branch December 9, 2019 13:51
arikfr added a commit that referenced this pull request Dec 11, 2019
* Remove app/service/query-string (unused) and its dependency.

* Fix usage of mixed operators.

* eslint --fix fixes for missing dependencies for react hooks

* Fix: useCallback dependency passed to $http's .catch.

* Satisfy react/no-direct-mutation-state.

* Fix no-mixed-operators violations.

* Move the decision of whether to render Custom chart one level up to make sure hooks are called in the same order.

* Fix: name was undefined. It wasn't detected before because there is such global.

* Simplify eslint config and switch to creat-react-app's eslint base.

* Add prettier config.

* Make sure eslint doesn't conflict with prettier

* A few updates post eslint (#4425)

* Prettier command in package.json
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.

3 participants