Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove ts-ignore statements in the code base #3228

Open
palisadoes opened this issue Feb 17, 2025 · 1 comment
Open

Remove ts-ignore statements in the code base #3228

palisadoes opened this issue Feb 17, 2025 · 1 comment
Assignees
@yugal07
Labels
bug Something isn't working unapproved Unapproved for Pull Request

Comments

@palisadoes
Copy link
Contributor

Describe the bug

  1. There are ts-ignore statements in the code base
  2. This is poor coding practice
  3. We need to remove them and fix the code properly

To Reproduce

From the repository root directory issue this command:

grep --exclude-dir node_modules --exclude-dir docs -nr ts-ignore .

Expected behavior

  • No violations of good coding practice

Actual behavior

  • See above

Screenshots

Image

Additional details

  • N/A

Potential internship candidates

Please read this if you are planning to apply for a Palisadoes Foundation internship
@palisadoes palisadoes added the bug Something isn't working label Feb 17, 2025
@github-actions github-actions bot added the unapproved Unapproved for Pull Request label Feb 17, 2025
@palisadoes palisadoes mentioned this issue Feb 17, 2025
Closed
@yugal07
Copy link

yugal07 commented Feb 17, 2025

I will take on this @palisadoes

@yugal07 yugal07 mentioned this issue Feb 19, 2025
Open
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yugal07 yugal07

Labels
bug Something isn't working unapproved Unapproved for Pull Request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@palisadoes @yugal07