Skip to content

Conversation

@DavidVujic
Copy link
Owner

@DavidVujic DavidVujic commented Nov 8, 2025

Description

The poly tool now detects circular dependencies between bricks, to avoid any possible runtime errors.

The poly deps and poly deps --brick <the brick> now reports on bricks that are both used by and used in a brick.

The notification is information, the command will not return with a failed exit code. The reason is that circular brick dependencies can work without runtime errors.

Motivation and Context

Solves feature request #394

How Has This Been Tested?

✅ CI
✅ Unit tests
✅ Local run of the command on the polylith repo itself containing two circular brick dependencies (see #393)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

@DavidVujic DavidVujic changed the title wip(deps): identify circular dependencies feat(deps): identify circular brick dependencies Nov 9, 2025
@DavidVujic DavidVujic marked this pull request as ready for review November 9, 2025 11:53
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 9, 2025

@DavidVujic DavidVujic merged commit e9db163 into main Nov 9, 2025
4 checks passed
@DavidVujic DavidVujic deleted the 394-deps-circular branch November 9, 2025 15:12
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