Skip to content

Conversation

oliver-sanders
Copy link
Member

@oliver-sanders oliver-sanders commented Sep 4, 2025

Improve broadcast documentation and cover the broadcast expiry change in cylc/cylc-flow#6964

Requirements check-list

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.

@oliver-sanders oliver-sanders added this to the 8.6.x milestone Sep 4, 2025
@oliver-sanders oliver-sanders self-assigned this Sep 4, 2025
@oliver-sanders oliver-sanders marked this pull request as ready for review September 4, 2025 16:00
Copy link
Member

@hjoliver hjoliver left a comment

Choose a reason for hiding this comment

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

Good, a few minor changes suggested.

Copy link
Member

@hjoliver hjoliver left a comment

Choose a reason for hiding this comment

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

A few more tweaks suggested.

@MetRonnie MetRonnie self-requested a review September 12, 2025 13:06
Co-authored-by: Hilary James Oliver <[email protected]>
Copy link
Member

@hjoliver hjoliver left a comment

Choose a reason for hiding this comment

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

Approval subject to two tweaks.

@wxtim wxtim requested review from wxtim and removed request for MetRonnie September 19, 2025 11:33
Copy link
Member

@wxtim wxtim left a comment

Choose a reason for hiding this comment

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

Alles Gut. Please take @hjoliver and my suggestions.

Co-authored-by: Hilary James Oliver <[email protected]>
Co-authored-by: Tim Pillinger <[email protected]>
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.

3 participants