Skip to content

Conversation

@thatguyinabeanie
Copy link
Collaborator

@thatguyinabeanie thatguyinabeanie commented Sep 5, 2025

Description

Context, Consequences, & Considerations

Required: Please step through the following list, pausing at each item to consider your change in relation to the item's context.
Check the box to mark that it applies, and enter your relevant notes under the item.

  • Security: This has security implications. This includes (but not limited to) adding users, modifying user/app permissions, network rules/policies, changing a system interconnection, or changing an authorization strategy.
    • This PR does not require security review. These changes are part of a project plan that has already undergone security review. The link is provided below.
    • This PR requires security review. Add the security label to this PR then request a review from the Security Code Reviewers Team.
  • Execution: This change requires commands to be run outside of the normal merge.
  • Impact: This change may cause service interruptions.
  • Testing: How did you test this change (unit tests, acceptance tests, etc.)? Did you do any manual testing?

manual

  • Testing: How will you confirm this change once it's merged?
  • Documentation: Documentation to reflect this change has been added to Confluence or Zendesk.
  • All items of the checklist have been considered and this PR description is complete.

Copy link

@leanne73 leanne73 left a comment

Choose a reason for hiding this comment

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

lgtm

@thatguyinabeanie thatguyinabeanie merged commit 6059d78 into master Sep 5, 2025
@civis-robot
Copy link

Pull Request Merged!!! This build is running now.

@thatguyinabeanie thatguyinabeanie deleted the civis-11444-just-need-to-rebuild-base-image branch September 5, 2025 15:11
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