Skip to content

Conversation

@tomassedovic
Copy link
Contributor

@tomassedovic tomassedovic commented Jan 23, 2026

@tomassedovic
Copy link
Contributor Author

@obi1kenobi I've opened a draft of for cargo-semver-checks in 2026.

It's mostly updating it to the new template and I've removed sections that seemed to me like they were done.

But I found it hard to summarise the status quo section. Would you please take a look at this and help us filling it in? Also, please if I cut too much, let us know!

@tomassedovic tomassedovic marked this pull request as draft January 23, 2026 16:47
Copy link
Member

@obi1kenobi obi1kenobi left a comment

Choose a reason for hiding this comment

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

Thank you so much for putting this together! I think it looks great, and I added some suggestions for the TODO and a couple of other places in case readers and reviewers want more detail.

Pinging a couple of T-rustdoc folks to see how they want to play things from their end. Whenever they feel this is good to go, I'm onboard too.

Thanks again!


**Owner:** @obi1kenobi, as maintainer of `cargo-semver-checks`

I (@obi1kenobi) will be working on this effort. The only other resource request would be occasional discussions and moral support from the [cargo] and [rustdoc] teams, of which I already have the privilege as maintainer of a popular cargo plugin that makes extensive use of rustdoc JSON.
Copy link
Member

Choose a reason for hiding this comment

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

@aDotInTheVoid how would you feel about mentioning the cross-crate rustdoc JSON work you've been doing here, to expand slightly from merely "moral support"?

Also @GuillaumeGomez would it be prudent to explicitly request T-rustdoc review capacity for things like rust-lang/rust#148379 or is that fine under "moral support"? I expect that PR is 95% of what we need, and I only expect a couple of tiny follow-up PRs after it (e.g. implied "outlives" bounds for lifetimes, which is fairly simple to do). So I feel a bit silly asking explicitly for PR review capacity when the bulk of the peer-reviewing might be done before this project goal gets merged :) Your call!

Copy link
Member

Choose a reason for hiding this comment

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

Yeah I think it's fine.

Comment on lines +77 to +78
| Implement type-checking in lints | | |
| Implement linting across crate boundaries | | |
Copy link
Member

Choose a reason for hiding this comment

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

I'm fine with this, but just noting that these are both huge tasks and the most likely outcome is that we make significant progress in 2026h1 and yet both boxes remain unchecked and spill over into 2026h2 :)

Hopefully nobody judges this task based on just the boxes in this table, especially when the 2025 project summary is 5000 words and shows the project is literally growing exponentially!

But just flagging it as a concern — and if you think it's worth it, I can try to come up with subtasks so it's easier to demonstrate progress merely with the checkboxes. Your call!

Copy link
Contributor

Choose a reason for hiding this comment

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

It's fine as is, imo.

Co-authored-by: Predrag Gruevski <2348618+obi1kenobi@users.noreply.github.com>
Comment on lines +77 to +78
| Implement type-checking in lints | | |
| Implement linting across crate boundaries | | |
Copy link
Contributor

Choose a reason for hiding this comment

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

It's fine as is, imo.

@nikomatsakis nikomatsakis marked this pull request as ready for review January 29, 2026 15:50
@nikomatsakis nikomatsakis merged commit 6f89ea5 into rust-lang:main Jan 29, 2026
4 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.

4 participants