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 not used code #52

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

Remove not used code #52

wants to merge 11 commits into from

Conversation

MrSuddenJoy
Copy link
Owner

@MrSuddenJoy MrSuddenJoy commented Jun 19, 2024 •
edited
Loading

Closes #51

Details

Many of the code is of type legacy introduced by either @adam-boduch and/or @danon

@MrSuddenJoy MrSuddenJoy added Code needs refactoring Suggests code that needs refactoring DeepSource Issue created with/by DeepSource app labels Jun 19, 2024
@MrSuddenJoy MrSuddenJoy added this to the Refactoring code-base milestone Jun 19, 2024
@deepsource-io deepsource.io
Copy link
Contributor

deepsource-io bot commented Jun 19, 2024 •
edited
Loading

Here's the code health analysis summary for commits dd2f4bf..709fb6c. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource PHP LogoPHP✅ SuccessView Check ↗
DeepSource JavaScript LogoJavaScript✅ Success
🎯 5 occurences resolved
View Check ↗
💡 If you’re a repository administrator, you can configure the quality gates from the settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Code needs refactoring Suggests code that needs refactoring DeepSource Issue created with/by DeepSource app
Projects
None yet
Milestone
Refactoring code-base
Development

Successfully merging this pull request may close these issues.

(JS-0356) Found unused variables in TypeScript code
1 participant
@MrSuddenJoy