Skip to content

chore: Make cargo doc fail on warnings - #1277

Open
Techassi wants to merge 2 commits into
mainfrom
chore/make-cargo-doc-fail-on-warnings
Open

Techassi wants to merge 2 commits into
mainfrom
chore/make-cargo-doc-fail-on-warnings

Conversation

@Techassi

@Techassi Techassi commented Sep 16, 2026

Copy link
Copy Markdown
Member

Triggered by stackabletech/issues#848 (comment), similar to stackabletech/operator-templating#650.

cargo doc hooks run by prek now error on warnings.

@Techassi Techassi self-assigned this Sep 16, 2026
@Techassi Techassi added the release-retro/26.7.0 Indicates this issue or PR is part of improvements resulting from the Release Retro 26.7.0 label Sep 16, 2026
@Techassi Techassi moved this to Development: In Progress in Stackable Engineering Sep 16, 2026
@Techassi Techassi moved this from Development: In Progress to Development: Waiting for Review in Stackable Engineering Sep 16, 2026

@NickLarsenNZ NickLarsenNZ 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.

Hmm, I find warnings useful for deprecating things without a breaking change.
I'll approve it since we already do that, but we should maybe look at relaxing that rule (it could be a separate check that is allowed to fail so at least there is feedback).

@NickLarsenNZ NickLarsenNZ moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Sep 21, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-retro/26.7.0 Indicates this issue or PR is part of improvements resulting from the Release Retro 26.7.0

Projects

Status: Development: In Review

Development

Successfully merging this pull request may close these issues.

2 participants