Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

chore(website): error viewer in playground#5061

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
bradzacher merged 14 commits intomainfromchore/errors-view-in-playground
May 26, 2022

Conversation

armano2
Copy link
Collaborator

@armano2armano2 commentedMay 23, 2022
edited
Loading

PR Checklist

Overview

Supper basic design for listing all errors when no ast viewer is selected, prints both ts and eslint errors (and any other errors reported by editor)

Changes

  • playground by default displays all errors in right panel with ability to fix fixable eslint errors
  • suggestions are now correctly displayed in Fix me in editor

Demo

image

JoshuaKGoldberg reacted with hooray emoji
@armano2armano2 added the DO NOT MERGEPRs which should not be merged yet labelMay 23, 2022
@nx-cloud
Copy link

nx-cloudbot commentedMay 23, 2022
edited
Loading

☁️ Nx Cloud Report

CI is running/has finished running commands for commit3694da2. 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.

@typescript-eslint
Copy link
Contributor

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 onhttps://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.

@armano2armano2 marked this pull request as draftMay 23, 2022 20:01
@netlify
Copy link

netlifybot commentedMay 23, 2022
edited
Loading

Deploy Preview fortypescript-eslint ready!

NameLink
🔨 Latest commit3694da2
🔍 Latest deploy loghttps://app.netlify.com/sites/typescript-eslint/deploys/628f8ef7fe53d900084a2a5a
😎 Deploy Previewhttps://deploy-preview-5061--typescript-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@armano2armano2 added package: websiteIssues related to the @typescript-eslint website enhancementNew feature or request labelsMay 23, 2022
@bradzacher
Copy link
Member

this is so good.

leoclap

@armano2
Copy link
CollaboratorAuthor

armano2 commentedMay 23, 2022
edited
Loading

@bradzacher
Copy link
Member

Just do it here if you like!

armano2 reacted with thumbs up emoji

@armano2armano2 changed the titlePOC: for error viewer in playgroundPOC: error viewer in playgroundMay 24, 2022
@armano2
Copy link
CollaboratorAuthor

most of functionalities seem to be working now, any ideas how fixers should look like?

@bradzacher
Copy link
Member

any ideas how fixers should look like?

I'm okay with having the button on the message like you've currently got.
Maybe make the text say "Fix" instead of "apply"

image

Do you have handling for suggestion fixers?
Maybe if it has suggestions then it can be a split button eg:
image

@armano2
Copy link
CollaboratorAuthor

armano2 commentedMay 24, 2022
edited
Loading

yes, suggestions and fixers are joined together, you can see an example hereDemo I was thinking about adding dropdown, but, its going to be even harder to make it work well in mobile (witch is most likely next)

strict-boolean-expressions has only suggestions

bradzacher reacted with thumbs up emoji

@armano2armano2 changed the titlePOC: error viewer in playgroundchore(website): error viewer in playgroundMay 24, 2022
@armano2armano2 marked this pull request as ready for reviewMay 24, 2022 22:24
@armano2armano2 removed the DO NOT MERGEPRs which should not be merged yet labelMay 25, 2022
@JoshuaKGoldberg
Copy link
Member

What an odd Netlify failure...https://app.netlify.com/sites/typescript-eslint/deploys/628d85ad0e4cb800089c5202

9:27:08 PM: [3/5] Fetching packages...9:27:09 PM: error Couldn't find a package.json file in "/opt/build/repo/packages/utils"

JoshuaKGoldberg
JoshuaKGoldberg previously approved these changesMay 25, 2022
Copy link
Member

@JoshuaKGoldbergJoshuaKGoldberg left a comment
edited
Loading

Choose a reason for hiding this comment

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

👏 Glorious!! This is going to be awesome to have in prod!

JoshuaKGoldberg
JoshuaKGoldberg previously approved these changesMay 25, 2022
Copy link
Member

@JoshuaKGoldbergJoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

Awesome!! 👏

bradzacher
bradzacher previously approved these changesMay 26, 2022
Copy link
Member

@bradzacherbradzacher left a comment

Choose a reason for hiding this comment

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

sobeautiful.gif

Co-authored-by: Josh Goldberg <me@joshuakgoldberg.com>
@bradzacherbradzacher merged commiteaa5d7b intomainMay 26, 2022
@bradzacherbradzacher deleted the chore/errors-view-in-playground branchMay 26, 2022 15:34
@github-actionsgithub-actionsbot locked asresolvedand limited conversation to collaboratorsJun 26, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers

@bradzacherbradzacherbradzacher approved these changes

@JoshuaKGoldbergJoshuaKGoldbergJoshuaKGoldberg left review comments

Assignees
No one assigned
Labels
enhancementNew feature or requestpackage: websiteIssues related to the @typescript-eslint website
Projects
No open projects
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Bug(website): compiler options are not passed to the lint run
3 participants
@armano2@bradzacher@JoshuaKGoldberg

[8]ページ先頭

©2009-2025 Movatter.jp