Skip to content

build: add sensible default for WORKSPACE#1349

Merged
joaodinissf merged 2 commits into
dsldevkit:masterfrom
joaodinissf:build/default-workspace-property
May 19, 2026
Merged

build: add sensible default for WORKSPACE#1349
joaodinissf merged 2 commits into
dsldevkit:masterfrom
joaodinissf:build/default-workspace-property

Conversation

@joaodinissf
Copy link
Copy Markdown
Collaborator

@joaodinissf joaodinissf commented May 18, 2026

What

<workspace> in ddk-parent now defaults to ${maven.multiModuleProjectDirectory} (= repo root) when the WORKSPACE env var is unset. CI keeps setting WORKSPACE explicitly; that path is unchanged.

Drops the export WORKSPACE=… instruction from AGENTS.md — local devs no longer need it.

Register tycho-build as a Maven core extension so `mvn -pl :<module> -am`
walks Require-Bundle / Import-Package when computing the reactor —
without it, only Maven <dependency> blocks are considered.

Version pinned via .mvn/maven.config to track ddk-parent's tycho.version.

Incompatible with mvnd; use plain mvn for -am.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@joaodinissf joaodinissf force-pushed the build/default-workspace-property branch from 121328b to f757aec Compare May 18, 2026 16:53
@joaodinissf joaodinissf marked this pull request as ready for review May 18, 2026 16:54
@joaodinissf joaodinissf enabled auto-merge (rebase) May 18, 2026 16:54
`<workspace>` defaults to ${maven.multiModuleProjectDirectory} when the
WORKSPACE env var is unset. CI's env-var path is unchanged.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@joaodinissf joaodinissf force-pushed the build/default-workspace-property branch from f757aec to da6f075 Compare May 18, 2026 16:56
@joaodinissf joaodinissf merged commit 5efde17 into dsldevkit:master May 19, 2026
4 checks passed
@joaodinissf joaodinissf deleted the build/default-workspace-property branch May 19, 2026 07:05
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