Skip to content

Conversation

bernardjkim
Copy link
Contributor

Supports: #51682, #55075
This PR updates the automatic review guide to include steps to demonstrate how to setup time-based rules. This also updates the access_monitoring_rules references page to update YAML spec documentation.

@bernardjkim bernardjkim added the no-changelog Indicates that a PR does not require a changelog entry label Oct 15, 2025
Copy link

github-actions bot commented Oct 15, 2025

Amplify deployment status

Branch Commit Job ID Status Preview Updated (UTC)
bernard/docs-automatic-review-schedules HEAD 1 ✅SUCCEED bernard-docs-automatic-review-schedules 2025-10-15 22:38:59

Copy link
Contributor

@ptgott ptgott Oct 16, 2025

Choose a reason for hiding this comment

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

I think the new H2 might be a little buried, and that we can work the different automatic review sections into the flow of the guide so users don't skip over them. We can make it clearer that the workflow doesn't end with adding traits.

I would:

  • Make Step 3/3 something like "Configure automatic reviews"
  • Turn all the current H2s ending with "automatic reviews" into H3s of the new Step 3
  • Add an intro paragraph to Step 3 indicating how automatic reviews make use of the steps the user has already completed, explain what the user has left to do: configure one or more kinds of automatic review.

access_request.spec.resource_labels_intersection["env"].contains("dev")
desired_state: reviewed
automatic_review:
integration: builtin
Copy link
Contributor

Choose a reason for hiding this comment

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

builtin is a name of the default review integration?

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

Labels

documentation no-changelog Indicates that a PR does not require a changelog entry size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants