Skip to content

Bom versions for maven-war-plugin in all POMs.xml#11413

Merged
offtherailz merged 5 commits intogeosolutions-it:masterfrom
stefanocudini:fix-pom-template-version
Sep 2, 2025
Merged

Bom versions for maven-war-plugin in all POMs.xml#11413
offtherailz merged 5 commits intogeosolutions-it:masterfrom
stefanocudini:fix-pom-template-version

Conversation

@stefanocudini
Copy link
Copy Markdown
Member

@stefanocudini stefanocudini commented Aug 29, 2025

Description

Fix issue #11415
updated all pom.xml using bom versions for Maven plugin maven-war-plugin
moved maven-war-plugin.version define in root pom.xml and referred in web/pom.xml
A new Build process tested.

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

@stefanocudini stefanocudini requested review from MV88 and tdipisa August 29, 2025 07:30
@stefanocudini stefanocudini self-assigned this Aug 29, 2025
Copy link
Copy Markdown
Member

@tdipisa tdipisa left a comment

Choose a reason for hiding this comment

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

@stefanocudini also other POMs have that dependency. Please have a look.

@tdipisa tdipisa added this to the 2025.02.00 milestone Aug 29, 2025
Copy link
Copy Markdown
Contributor

@MV88 MV88 left a comment

Choose a reason for hiding this comment

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

hi @stefanocudini thanks for the pr there should be an issue attached to this and also it could be possible that this variable is needed elsewhere, can you do a quick research ?

image

@stefanocudini stefanocudini force-pushed the fix-pom-template-version branch from 5facafc to 8dfc74a Compare August 29, 2025 09:50
@stefanocudini stefanocudini linked an issue Aug 29, 2025 that may be closed by this pull request
6 tasks
@stefanocudini stefanocudini added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Aug 29, 2025
@stefanocudini stefanocudini requested a review from tdipisa August 29, 2025 10:16
@stefanocudini stefanocudini changed the title fix maven-war-plugin.version in root pom.xml bom versions for maven-war-plugin pom.xml Aug 29, 2025
@stefanocudini stefanocudini changed the title bom versions for maven-war-plugin pom.xml Bom versions for maven-war-plugin in all pom.xml Aug 29, 2025
@stefanocudini stefanocudini requested a review from tdipisa August 29, 2025 13:42
Copy link
Copy Markdown
Member

@tdipisa tdipisa left a comment

Choose a reason for hiding this comment

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

@stefanocudini there are conflicts to be fixed. Thank you

@tdipisa tdipisa self-requested a review September 1, 2025 12:29
@stefanocudini stefanocudini force-pushed the fix-pom-template-version branch from 832f29b to a8bcccc Compare September 1, 2025 13:00
@stefanocudini stefanocudini changed the title Bom versions for maven-war-plugin in all pom.xml Bom versions for maven-war-plugin in all POMs.xml Sep 1, 2025
@tdipisa tdipisa requested review from offtherailz and tdipisa and removed request for tdipisa September 1, 2025 16:28
@tdipisa tdipisa assigned offtherailz and unassigned stefanocudini Sep 1, 2025
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Sep 1, 2025
@offtherailz
Copy link
Copy Markdown
Member

@tdipisa please approve to allow merge.

@offtherailz offtherailz merged commit 0b58fee into geosolutions-it:master Sep 2, 2025
6 checks passed
dsuren1 added a commit to dsuren1/MapStore2 that referenced this pull request Sep 9, 2025
…tinerary

* commit 'ea419127df12f794a56f234f5660632511df7484':
  Update README.md
  Proj4 upgrade and include support for "Grid Based Datum Adjustments" geosolutions-it#11423 (geosolutions-it#11424)
  Bom versions for maven-war-plugin in all POMs.xml (geosolutions-it#11413)
  Fix geosolutions-it#11419 fixed group translation files (geosolutions-it#11420)
  Update geoserver.md (geosolutions-it#11399)
  Bump org.apache.tomcat:tomcat-coyote from 9.0.105 to 9.0.108 in /binary (geosolutions-it#11384)
  Update on the Swedish translation file for MapStore (geosolutions-it#11412)
  Fix geosolutions-it#11258 :For clipping/masking 3D tiles and terrain removed ClippingPlaneCollection and introduced ClippingPolygonCollection (geosolutions-it#11352)
  Fix : geosolutions-it#11400 Added quickFilters also  when exporting data from TableWidget (geosolutions-it#11401)
  Fix geosolutions-it#11404 Problem with user permissions still involving the "Save As" (geosolutions-it#11408)
  Fix geosolutions-it#11406 Pending changes prompt shows up after save as (geosolutions-it#11407)
  Fix geosolutions-it#11374 Improve the Street Smart plugin view for 3D maps visualization (geosolutions-it#11375)
  Update i18n showEmptyMessageGFI (geosolutions-it#11392)
  geosolutions-it#11369: Enhance non hyperlink query field to have spacing (geosolutions-it#11370)
  Bump org.apache.tomcat:tomcat-util from 9.0.105 to 9.0.107 in /binary (geosolutions-it#11305)
  chore: Remove element-closest dependency (geosolutions-it#11251)
  Fix the widget support for the WFS layer with no geom (geosolutions-it#11379)
  Support of widget in map even when layer have no geometry and fix time dependencies for the widget (geosolutions-it#11377)
  Performace Lag when there is a layer with very large geomerty geosolutions-it#11358 (geosolutions-it#11359)
  skip click on layer should return intersected features test on Cesium tests (geosolutions-it#11365)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Using bom versions for maven-war-plugin

5 participants