Skip to content

Almost at the end of main#1

Merged
groldan merged 5 commits intogroldan:geoserver3/java17from
aaime:geoserver3/java17
Jun 8, 2025
Merged

Almost at the end of main#1
groldan merged 5 commits intogroldan:geoserver3/java17from
aaime:geoserver3/java17

Conversation

@aaime
Copy link

@aaime aaime commented Jun 8, 2025

Hi @groldan while I kept on having a look at the errorprone situation, I fixed a few things. Approach:

  • Mass fix the whole Codebase if IntelliJ could do it
  • Ignore the warning otherwise, leaving each suppression for a rainy day (I typically fare better if I handle one at a time, it's always the same thing to fix).

Not sure if you'll find it useful or not, take it as is, throw it away, cherry pick parts you like... do as you see fit! 😄

Checklist

For core and extension modules:

  • New unit tests have been added covering the changes.
  • Documentation has been updated (if change is visible to end users).
  • There is an issue in GeoTools Jira (except for changes not visible to end users).
  • Commit message(s) must be in the form [GEOT-XYZW] Title of the Jira ticket.
  • Bug fixes and small new features are presented as a single commit.
  • The commit targets a single objective (if multiple focuses cannot be avoided, each one is in its own commit, and has a separate ticket describing it).

@aaime aaime changed the base branch from main to geoserver3/java17 June 8, 2025 14:45
@groldan groldan merged commit 8be7e70 into groldan:geoserver3/java17 Jun 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants