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

Fix: Eliminated unnecessary code #283

Merged
merged 2 commits into from
Apr 24, 2024
Merged

Fix: Eliminated unnecessary code #283

merged 2 commits into from
Apr 24, 2024

Conversation

UO283615
Copy link
Contributor

This code was written in case we were to add distractors in different ways, but we did not so it is better to remove it

@UO283615 UO283615 added the backend ⚙️ Backend issue label Apr 24, 2024
@UO283615 UO283615 added this to the Final presentation milestone Apr 24, 2024
@UO283615 UO283615 self-assigned this Apr 24, 2024
@jjgancfer jjgancfer self-requested a review April 24, 2024 21:18
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@Toto-hitori Toto-hitori merged commit 89f7626 into develop Apr 24, 2024
2 checks passed
@Toto-hitori Toto-hitori deleted the fix/codeSmells branch April 25, 2024 15:47
This was referenced Apr 27, 2024
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Toto-hitori Toto-hitori Toto-hitori approved these changes

@jjgancfer jjgancfer jjgancfer approved these changes

Assignees

@UO283615 UO283615

Labels
backend ⚙️ Backend issue
Projects
None yet
Milestone
Final presentation
Development

Successfully merging this pull request may close these issues.
3 participants
@UO283615 @jjgancfer @Toto-hitori