fixes issue with playground not respecting jsx options
web-linter parsed code but you could see error in editor
const x = <Boo>this.foo();
Note: changing from and to JSX support in editor requires browser refresh, this is limitation of sandbox
allow user-definedcompilerOptions to be passed to WebLinter
regression fix for bug introduced inchore(website): rewrite WebLinter to typescript and fix support for ts 4.7 #5034

Test cases

fix(website): pass user defined compilerOptions to linter

4b73a94

armano2 added bug

Something isn't working

package: websiteIssues related to the @typescript-eslint website labels

May 25, 2022

Copy link

Contributor

typescript-eslintbot commentedMay 25, 2022

Thanks for the PR,@armano2!

typescript-eslint is a 100% community driven project, and we are incredibly grateful that you are contributing to that community.

The core maintainers work on this in their personal time, so please understand that it may not be possible for them to review your work immediately.

Thanks again!

🙏Please, if you or your company is finding typescript-eslint valuable, help us sustain the project by sponsoring it transparently on https://opencollective.com/typescript-eslint. As a thank you, your profile/company logo will be added to our main README which receives thousands of unique visitorsper day.

Copy link

nx-cloudbot commentedMay 25, 2022•
edited
Loading

☁️ Nx Cloud Report

CI is running/has finished running commands for commit38bbef9. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

✅ Successfully ran 47 targets

Sent with 💌 fromNxCloud.

armano2 assignedbradzacher

May 25, 2022

Copy link

netlifybot commentedMay 25, 2022•
edited
Loading

✅ Deploy Preview fortypescript-eslint ready!

Name	Link
🔨 Latest commit	`38bbef9`
🔍 Latest deploy log	https://app.netlify.com/sites/typescript-eslint/deploys/628f9f75a0bcb1000ab52d04
😎 Deploy Preview	https://deploy-preview-5080--typescript-eslint.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to yourNetlify site settings.

armano2 unassignedbradzacher

May 25, 2022

armano2 requested a review frombradzacher

May 25, 2022 18:43

fix: normalize CompilerOptions defaults

128debf

bradzacher previously approved these changes

May 26, 2022

View reviewed changes

Copy link

Member

bradzacher left a comment

Choose a reason for hiding this comment

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

One quick request - could we makestrict: true the default?
Or at leaststrictNullChecks.

Otherwise

Copy link

CollaboratorAuthor

armano2 commentedMay 26, 2022•
edited
Loading

One quick request - could we makestrict: true the default? Or at leaststrictNullChecks.

sadly no, we can't really define defaults, user is not going to be able to disable them for now, changing defaults requires changes to be done in config editor, but that will be done when i get to improving config editor

Config editor currently removes all options that are set to false (yes this is bad, but that's how it was done)

added to backlog,Website Playground (view)