Skip to content

Update SMS branch protection match pattern#516

Merged
mnasiadka merged 1 commit into
mainfrom
sms-branch-protection
May 18, 2026
Merged

Update SMS branch protection match pattern#516
mnasiadka merged 1 commit into
mainfrom
sms-branch-protection

Conversation

@Alex-Welsh
Copy link
Copy Markdown
Member

No description provided.

@Alex-Welsh Alex-Welsh requested a review from a team as a code owner May 18, 2026 08:46
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the branch protection pattern for the SMSLab repository from a broad wildcard to a specific versioning format (smslab/202[0-9].[1-2]). Feedback suggests that the new pattern is overly restrictive as it lacks a trailing wildcard, which would exclude patch versions or stable suffixes. It is recommended to use a broader pattern like smslab/202* in both the Terraform configuration and the import script to ensure better maintainability and coverage of all release branches.

Comment thread terraform/github/branches.tf
Comment thread terraform/github/import_resources.py
@stackhpc-ci
Copy link
Copy Markdown
Contributor

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖success

Validation Output

Success! The configuration is valid.


Terraform Plan 📖success

Show Plan

undefined

Pusher: @Alex-Welsh, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub

@Alex-Welsh Alex-Welsh mentioned this pull request May 18, 2026
@mnasiadka mnasiadka merged commit 8b3c9ca into main May 18, 2026
5 checks passed
@mnasiadka mnasiadka deleted the sms-branch-protection branch May 18, 2026 14:30
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