Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Refactor: Remove unused imports across multiple frontend components #6232

Merged
merged 4 commits into from
Feb 18, 2025

Conversation

ogabrielluiz
Copy link
Contributor

Ran make format_frontend with organizeImportsSkipDestructiveCodeActions: false,.

Eliminate unnecessary imports in various frontend components to improve code cleanliness and maintainability.

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Feb 10, 2025
Copy link
Contributor

@anovazzi1 anovazzi1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 11, 2025
@Cristhianzl Cristhianzl added this pull request to the merge queue Feb 18, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 18, 2025
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Feb 18, 2025
@ogabrielluiz ogabrielluiz added this pull request to the merge queue Feb 18, 2025
Copy link

codspeed-hq bot commented Feb 18, 2025

CodSpeed Performance Report

Merging #6232 will improve performances by 14.49%

Comparing organize-imports-js (6649fbd) with main (45bac37)

Summary

⚡ 1 improvements
✅ 13 untouched benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
test_build_flow_invalid_job_id 14.8 ms 12.9 ms +14.49%

Merged via the queue into main with commit b8346a0 Feb 18, 2025
35 of 41 checks passed
@ogabrielluiz ogabrielluiz deleted the organize-imports-js branch February 18, 2025 20:09
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants