Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Update comment.ts #885

Conversation

AlooDon
Copy link

@AlooDon AlooDon commented Nov 16, 2023

It seems like the ESLint is still producing a warning for the variable _unusedKey being defined but never used. To address this warning in ESLint, you can disable the @typescript-eslint/no-unused-vars rule for that specific line using an eslint comment:

By commenting out _unusedKey within the array destructuring, you are essentially telling ESLint to ignore this variable for the purpose of the no-unused-vars rule.

Resolves #870
Resolves #118
Quality Assurance:
Screenshot from 2023-11-16 13-12-31

/wallet - 0x38241902297144Df7C30F7De66963059dB143da2

It seems like the ESLint is still producing a warning for the variable _unusedKey being defined but never used. To address this warning in ESLint, you can disable the @typescript-eslint/no-unused-vars rule for that specific line using an eslint comment:

By commenting out _unusedKey within the array destructuring, you are essentially telling ESLint to ignore this variable for the purpose of the no-unused-vars rule.
Copy link

netlify bot commented Nov 16, 2023

Deploy Preview for ubiquibot-staging ready!

Name Link
🔨 Latest commit f521f6c
🔍 Latest deploy log https://app.netlify.com/sites/ubiquibot-staging/deploys/6555c90128f6d40008727048
😎 Deploy Preview https://deploy-preview-885--ubiquibot-staging.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 your Netlify site configuration.

@AlooDon
Copy link
Author

AlooDon commented Nov 16, 2023

@pavlovcik @sergfeldman @zgorizzo69 @Venoox can someone review this!!

@0x4007
Copy link
Member

0x4007 commented Nov 16, 2023

Thank you for your contribution. However we paused public contribution on this repository due to a large refactoring that affects almost every file occurring now. It is likely that we will not be able to use any of your changes.

@0x4007 0x4007 closed this Nov 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix check annotation on ci run Test issue
2 participants