Skip to content

Remove star imports #1057

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
Closed
SquidXTV opened this issue Mar 16, 2024 · 0 comments · Fixed by #1092
Closed

Remove star imports #1057

SquidXTV opened this issue Mar 16, 2024 · 0 comments · Fixed by #1092
Labels
bug Something isn't working priority: low

Comments

@SquidXTV
Copy link
Member

About

image
We currently have a lot of star imports in the project.
They should be replaced by normal imports and maybe add a check to sonar to prevent this from happening.
@SquidXTV SquidXTV added bug Something isn't working good first issue Good for newcomers priority: low and removed good first issue Good for newcomers labels Mar 16, 2024
@Together-Java Together-Java deleted a comment from SquidXTV Mar 20, 2024
@Taz03 Taz03 assigned Taz03 and unassigned Taz03 Mar 20, 2024
SimonVerhoeven added a commit to SimonVerhoeven/TJ-Bot that referenced this issue Apr 16, 2024
@SimonVerhoeven
chore: remove star imports (resolves Together-Java#1057)
1da7aae
@SimonVerhoeven SimonVerhoeven mentioned this issue Apr 16, 2024
Merged
tj-wazei pushed a commit that referenced this issue May 12, 2024
@SimonVerhoeven
Remove star imports (#1092)
d198f19 
chore: remove star imports (resolves #1057)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working priority: low
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove star imports SimonVerhoeven/TJ-Bot
2 participants
@Taz03 @SquidXTV