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.

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

[Backport 2.x] Forbid wildcard imports (#1180) #1183

Merged
merged 1 commit into from
Sep 5, 2024

Conversation

Xtansia
Copy link
Collaborator

@Xtansia Xtansia commented Sep 4, 2024

Description

Backport #1180 via e856ecb

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

* Forbid wildcard imports

Needs to be implemented as an throwing custom step as Spotless doesn't have built in support to remove/deny wildcards: diffplug/spotless#649

Signed-off-by: Thomas Farr <[email protected]>

* Make conventions plugin

Signed-off-by: Thomas Farr <[email protected]>

* Fix wildcard imports

Signed-off-by: Thomas Farr <[email protected]>

* Allow overriding eclipse formatter config file

Signed-off-by: Thomas Farr <[email protected]>

* Fix message formatting

Signed-off-by: Thomas Farr <[email protected]>

---------

Signed-off-by: Thomas Farr <[email protected]>
(cherry picked from commit e856ecb)
@Xtansia Xtansia changed the title Forbid wildcard imports (#1180) [Backport 2.x] Forbid wildcard imports (#1180) Sep 4, 2024
@reta reta merged commit 2d6d5f8 into opensearch-project:2.x Sep 5, 2024
57 of 58 checks passed
@Xtansia Xtansia deleted the backport/backport-1180-to-2.x branch September 5, 2024 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants