Skip to content

chore: simplify mta.yaml - #47

Open
lisajulia wants to merge 1 commit into
mainfrom
simplify-sidecar-yaml
Open

chore: simplify mta.yaml#47
lisajulia wants to merge 1 commit into
mainfrom
simplify-sidecar-yaml

Conversation

@lisajulia

@lisajulia lisajulia commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Fix and Simplify mta.yaml Configurations for Sidecar Projects

This PR makes small but important corrections to the mta.yaml files for both the java-bookshop and node-bookshop sidecar test projects.

Changes in tests/sidecar-projects/java-bookshop/mta.yaml:

  • Simplified the HANA build step by replacing multiple commands (build, cleanup, directory creation, file copying) with a single command using --dest gen/db flag:
    - npx --yes cds build --for hana --dest gen/db
  • Updated build-result for the sidecar-db-deployer module from gen/ to gen/db to match the new output path.

Changes in tests/sidecar-projects/node-bookshop/mta.yaml:

  • Fixed a copy-paste error where agent-java-bookshop-aicore was incorrectly referenced in the node-bookshop configuration — corrected to agent-node-bookshop-aicore in both the module requires section and the resources definition.
  • Minor indentation/formatting fixes.

Category: 🐛 Bug Fix / 🔧 Chore

Have you...

  • Added relevant entry to the change log?

  • 🔄 Regenerate and Update Summary

PR Bot Information

Version: 1.29.33

  • Correlation ID: 3bb7f560-9d67-11f1-9c84-51d4d1057b77
  • Summary Prompt: Default Prompt
  • LLM: anthropic--claude-4.6-sonnet
  • Output Template: Repository PR Template
  • File Content Strategy: Full file content
  • Event Trigger: pull_request.opened

@lisajulia
lisajulia requested a review from Akatuoro August 21, 2026 13:50
@lisajulia
lisajulia requested a review from a team as a code owner August 21, 2026 13:50
@lisajulia lisajulia changed the title Small changes to mta.yaml chore: simplify mta.yaml Aug 21, 2026
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