Skip to content

Fixes incorrect references to crate::shutdown::Shutdown in documentation - #14

Merged
martynp merged 1 commit into
mainfrom
6-guide-links-to-non-existent-configshutdown
Jul 5, 2026
Merged

martynp merged 1 commit into
mainfrom
6-guide-links-to-non-existent-configshutdown

Conversation

@martynp

@martynp martynp commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@martynp martynp linked an issue Jul 5, 2026 that may be closed by this pull request
@martynp
martynp requested a review from Copilot July 5, 2026 13:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects documentation references for graceful shutdown configuration, ensuring links point to the actual ShutdownConfig type (and its “Triggers” section) instead of an incorrect Shutdown reference, and aligns the README’s stated MSRV with the workspace rust-version.

Changes:

  • Fixes the fairings guide link to point to shutdown::ShutdownConfig triggers on docs.rs.
  • Fixes an internal rustdoc link in Shutdown’s docs to reference ShutdownConfig#triggers.
  • Updates the README MSRV badge to 1.88.0+ to match Cargo.toml’s rust-version = "1.88.0".

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
website/docs/08-fairings.md Updates docs.rs link for “shutdown is triggered” to the correct ShutdownConfig#triggers target.
README.md Updates the MSRV badge to match the workspace rust-version.
core/lib/src/shutdown/handle.rs Fixes rustdoc link for “automatic triggers” to point to ShutdownConfig#triggers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@martynp
martynp merged commit ef5343b into main Jul 5, 2026
12 checks passed
@martynp
martynp deleted the 6-guide-links-to-non-existent-configshutdown branch July 5, 2026 16:17
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.

Guide links to non-existent config::Shutdown

2 participants