Conversation

YanaXu

Description

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

@CopilotCopilot AI review requested due to automatic review settings June 10, 2025 03:50
@azure-client-tools-bot-prdazure-client-tools-bot-prd
Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

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 introduces a new Actions workflow to automatically delete old Daily/Release_* branches older than 30 days, triggered on a schedule and via manual dis.

  • Adds a clean-branches.yml workflow with daily cron and manual dis triggers
  • Implements a PowerShell script that lists all branches matching Daily/Release_\d{14}, parses their embedded timestamp, and deletes those older than 30 days
  • Uses GH CLI to fetch repository info and perform branch deletions

@AzureAzure deleted a comment from Copilot AI Jun 10, 2025
@AzureAzure deleted a comment from Copilot AI Jun 10, 2025
@isra-fel

LGTM as long as we clear comments from copilot

@YanaXuYanaXu enabled auto-merge (squash) June 10, 2025 10:23

Choose a reason for hiding this comment

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

LGTM

@YanaXuYanaXu merged commit 8f34583 into main Jun 11, 2025
12 checks passed
@isra-felisra-fel deleted the yanxu/removebranches branch June 11, 2025 05:29
Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.