Skip to content

ES-1750 Update sonar-ps-plugin to LTA 2026.1 compatibility#2

Merged
Hannah Cartojano (hbcartojano) merged 8 commits into
masterfrom
ES-1750
May 7, 2026
Merged

ES-1750 Update sonar-ps-plugin to LTA 2026.1 compatibility#2
Hannah Cartojano (hbcartojano) merged 8 commits into
masterfrom
ES-1750

Conversation

@hbcartojano

@hbcartojano Hannah Cartojano (hbcartojano) commented Apr 20, 2026

Copy link
Copy Markdown

This updates the sonar-ps-plugin so it's compatible with SonarQube Server LTA 2026.1.

Dependency Updates

  • sonar-plugin-api: 10.11.0.246813.4.3.4290 (compatible with LTA 2026.1)
  • sonar-testing-harness: 10.7.0.9632726.1.0.118079 (compatible with LTA 2026.1)
  • commons-lang3: 3.12.03.18.0 (minimum to resolve issue)

Left comments below on file changes outside gretard#39
All tests pass and plugin works as intended in SonarQube Community 26.1.0.118079

@hbcartojano Hannah Cartojano (hbcartojano) changed the title ES-1750 Update sonar-ps-plugin to LTA 2026.2 compatibility ES-1750 Update sonar-ps-plugin to LTA 2026.1 compatibility Apr 21, 2026
Comment thread sonar-ps-plugin/.settings/org.eclipse.jdt.core.prefs
Comment thread sonar-ps-plugin/.settings/org.eclipse.jdt.core.prefs
Comment thread sonar-ps-plugin/.classpath
Comment thread sonar-ps-plugin/pom.xml
Comment thread sonar-ps-plugin/pom.xml
Comment thread sonar-ps-plugin/pom.xml Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Build and test using Docker Compose:
- Run tests: `docker compose run --rm test`
- Build package: `docker compose run --rm build`
- Use a custom Docker registry: `DEFAULT_DOCKER_REPO=your.registry.com/path docker compose run --rm test`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Let's not try to pre-empt what others are doing; this is a public repo and others can figure out what they need to do

Suggested change
- Use a custom Docker registry: `DEFAULT_DOCKER_REPO=your.registry.com/path docker compose run --rm test`

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This is mostly just so the next person at Iress knows what to do when they need to use our Docker private registry and update the plugin. It’s just being explicit, but I’m fine to remove it if needed.

Comment thread README.md Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants