Skip to content

Remove external sbt-dependency-graph plugin - #1307

Merged
pjfanning merged 1 commit into
apache:mainfrom
mkurz:issue-452-remove-sbt-dependency-graph
Sep 18, 2026
Merged

pjfanning merged 1 commit into
apache:mainfrom
mkurz:issue-452-remove-sbt-dependency-graph

Conversation

@mkurz

@mkurz mkurz commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Motivation

sbt has included dependency-tree support since 1.4. Pekko HTTP uses sbt 1.13.0, so the legacy net.virtual-void dependency is no longer needed.

The pull-request validator still relied on graph model classes supplied by that plugin.

Modification

  • Replace sbt-dependency-graph with addDependencyTreePlugin.
  • Read module IDs directly from sbt's public UpdateReport data.
  • Preserve dependency analysis across the Compile, Test, Runtime, Provided, and Optional configurations.
  • Preserve inclusion of the current project when checking changed modules.

Result

The build uses the dependency-tree implementation supplied for sbt 1.13.0 and no longer depends on the legacy plugin APIs.

Pull-request impact analysis retains the previous project-selection behavior.

Tests

  • sbt -batch "show sbtVersion" — passed; loaded the build and compiled 21 meta-build sources with sbt 1.13.0.
  • Scalafmt — passed.
  • git diff --check — passed.
  • Test suites not run per request.

Motivation:
The legacy sbt-dependency-graph plugin is no longer needed because dependency-tree support is available in sbt itself.

Modification:
Use addDependencyTreePlugin and derive PR validation module IDs directly from public UpdateReport data instead of legacy plugin graph types.

Result:
The build uses sbt 1.13 dependency-tree support while preserving PR impact analysis without obsolete plugin APIs.

Tests:
- sbt -batch "show sbtVersion" (passed; meta-build compiled with sbt 1.13.0)
- Test suites not run per request.

References:
Fixes apache#452
@mkurz

mkurz commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

Green.

@raboof raboof left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

seems like a nice cleanup!

@pjfanning pjfanning left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@pjfanning
pjfanning merged commit f5c2bb5 into apache:main Sep 18, 2026
6 checks passed
@mkurz
mkurz deleted the issue-452-remove-sbt-dependency-graph branch September 18, 2026 16:13
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.

Remove dependency on sbt-dependency-graph

4 participants