Skip to content

Add ability to ignore build-dependencies when checking for licenses#800

Merged
Jake-Shadle merged 1 commit intoEmbarkStudios:mainfrom
acpiccolo:feat/include-build-licenses
Jan 8, 2026
Merged

Add ability to ignore build-dependencies when checking for licenses#800
Jake-Shadle merged 1 commit intoEmbarkStudios:mainfrom
acpiccolo:feat/include-build-licenses

Conversation

@acpiccolo
Copy link
Copy Markdown
Contributor

@acpiccolo acpiccolo commented Oct 29, 2025

This PR (AI generated) for issue #744 adds a new include-build configuration option to the [licenses] section, allowing users to opt out of checking build-dependencies for license checks.

What changed:

src/licenses/cfg.rs: added include_build (TOML key: include-build) with default true and validation.
src/licenses/gather.rs: respect include-build (in combination with include-dev) when selecting which crates to check for licenses.
docs/src/checks/licenses/cfg.md: documented include-build.
Added a unit test and updated snapshots for deserialization.

Notes:

Default behavior preserved: build-dependencies are included by default to avoid breaking existing usage.

Please review and comment.

Resolves: #744

@Jake-Shadle Jake-Shadle force-pushed the feat/include-build-licenses branch from 254dbe5 to 35636f2 Compare January 8, 2026 11:08
@Jake-Shadle Jake-Shadle changed the title Add ability to ignore [build-dependencies] when checking for licenses #744 Add ability to ignore build-dependencies when checking for licenses Jan 8, 2026
@Jake-Shadle Jake-Shadle merged commit 04a5f8e into EmbarkStudios:main Jan 8, 2026
12 checks passed
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.

Add ability to ignore [build-dependencies] when checking for licenses

2 participants