feat: add method to obtain the map bounds #99
Conversation
| setEast(bounds.getNumber("east")); | ||
| } | ||
|
|
||
| public double getSouth() { |
There was a problem hiding this comment.
Use lombok @Getter and @Setter
There was a problem hiding this comment.
Lombok is not being used in the add-on. No need to replace setters/getters.
There was a problem hiding this comment.
If Lombok is not being used in the project, maybe it would be a good idea to remove the dependency. The only problem that I see is that it is a transitive dependency so it might cause issues in projects, so it might be considered a breaking change, so I think that we could implement the removal of the dependency in another PR later.
| } | ||
|
|
||
| @Override | ||
| public String toString() { |
There was a problem hiding this comment.
Annotate the class with@ToString(includeFieldNames=true) instead. See https://projectlombok.org/features/ToString
There was a problem hiding this comment.
Lombok is not being used in the add-on.
|
Kudos, SonarCloud Quality Gate passed!
|
Close #93