Skip to content

Remove not used code #52

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
Draft
MrSuddenJoy wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

Remove not used code #52

MrSuddenJoy 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
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