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

fixes a bug where flycheck was not showing some warnings for .tsx #157

Merged
merged 2 commits into from
Mar 10, 2022
Merged

fixes a bug where flycheck was not showing some warnings for .tsx #157

merged 2 commits into from
Mar 10, 2022

Conversation

LionyxML
Copy link
Contributor

@LionyxML LionyxML commented Mar 9, 2022

Full bug discussion here:
#156

Motivation

When developing inside .tsx files, we had missing warnings from flycheck due from some disabling, it looked like we could only (manually) have ts-tide or javascript eslint, not both enabled.

Approach

Inside frontside-javascript--tsx-web-mode-hook function we now disable all checkers and then add both 'tsx-tide 'javascript-eslint.

Alternate Designs

I am not sure how to do it another way, but this line might be optional:
(setq flycheck-disabled-checkers nil)

LionyxML and others added 2 commits March 9, 2022 19:28
It turns out that the checker chains are configurations that can be
set only once, and they'll affect all invocations of flycheck, so
instead of doing this in a web-mode hook, we can just do it once when
setting up frontside-javascript.
@cowboyd cowboyd merged commit f71adba into thefrontside:release Mar 10, 2022
@cowboyd
Copy link
Member

cowboyd commented Mar 11, 2022

Writing a test for this proved to be exceedingly difficult because it requires a full yarn install to work, so I just went ahead and merged. Also I found that we don't have to do this in a hook, we can do it once when we initially setup.

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.

2 participants