Skip to content

refactor: restructure sample-app into standalone Maven project#36

Merged
busehalis-sap merged 7 commits into
mainfrom
refactor/restructure-sample-app
Jul 1, 2026
Merged

refactor: restructure sample-app into standalone Maven project#36
busehalis-sap merged 7 commits into
mainfrom
refactor/restructure-sample-app

Conversation

@busehalis-sap

Copy link
Copy Markdown
Contributor

Summary

  • Renamed existing sample-app to integration-tests
  • Added a new minimal sample-app as a standalone Maven project
  • Replaced personal email with placeholder in CatalogServiceHandler
  • Enabled index page and explicit local mode in integration-tests application.yaml, removed commented-out remote service config

- Rename sample-app/ directory to integration-tests/ to better reflect its purpose
- Update root pom.xml module reference
- Update coverage-report/pom.xml artifact references and JaCoCo exec paths
- Update integration-tests pom artifactIds
- Update .github/actions/test/action.yml working-directory
- Update .github/actions/scan-with-sonar/action.yml exclusion path
@busehalis-sap busehalis-sap force-pushed the refactor/restructure-sample-app branch 2 times, most recently from edbd0fb to 63a5051 Compare June 30, 2026 16:54

@lisajulia lisajulia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Only one small change, otherwise lgtm. Thanks!

Comment thread integration-tests/srv/src/main/resources/application.yaml
@busehalis-sap busehalis-sap requested a review from lisajulia July 1, 2026 13:19
@busehalis-sap busehalis-sap merged commit 3373c0d into main Jul 1, 2026
6 checks passed
@busehalis-sap busehalis-sap deleted the refactor/restructure-sample-app branch July 1, 2026 13:58
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