-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
fix: show lint when unused variable is present in code #10599
fix: show lint when unused variable is present in code #10599
Conversation
This pull request is being automatically deployed with Vercel (learn more). π Inspect: https://vercel.com/get-appsmith/appsmith/GekxhkZNwpmuruGePxVmpqQLgnpD |
Unable to find test scripts. Please add necessary tests to the PR. |
4 similar comments
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
/ok-to-test sha=9ebfe5b |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/1740610742. |
Unable to find test scripts. Please add necessary tests to the PR. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/1740610742. Click to view performance test results
|
9ebfe5b
to
87b5528
Compare
/ok-to-test sha=87b5528 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/1747650493. |
Unable to find test scripts. Please add necessary tests to the PR. |
/ok-to-test sha=ba08502 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/1747667732. |
app/client/src/components/editorComponents/CodeEditor/lintHelpers.ts
Outdated
Show resolved
Hide resolved
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.
Looks good.
mentioned nit pick in above comment
ba08502
to
c2984e7
Compare
/ok-to-test sha=c2984e7 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/1758567186. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/1758567186. Click to view performance test results
|
/ok-to-test sha=cd2ff23 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/1758764000. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/1758764000. Click to view performance test results
|
Unable to find test scripts. Please add necessary tests to the PR. |
/ok-to-test sha=1588ace |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/1760248006. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/1760248006. Click to view performance test results
|
Description
This PR introduces warnings in linting. Users get warned when an unused variable is present in the code.
Fixes #10589
Type of change
How Has This Been Tested?
Manual
Checklist:
Test coverage results π§ͺ
π’ Total coverage has increased