-
Notifications
You must be signed in to change notification settings - Fork 929
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
quick fixes in editor #3928
quick fixes in editor #3928
Conversation
✅ Deploy Preview for remixproject ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
9c409ff
to
70cf359
Compare
import { default as fixes } from "./quickfixes" | ||
|
||
export class RemixCodeActionProvider implements monaco.languages.CodeActionProvider { | ||
props: EditorUIProps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use one space as a tab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the default for TS is 4 spaces
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#3939 sets all to 2 spaces. 1 is not best practice.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As linting is being fixed in different PR, can you review again? @LianaHus
Related to #3927
This PR introduces QuickFixes for:
More quick fixes will be added in upcoming PRs