Skip to content

Conversation

@mekarpeles
Copy link
Member

Closes #11135

Technical

Testing

Screenshot

Stakeholders

@Copilot Copilot AI review requested due to automatic review settings August 14, 2025 19:50
@github-actions github-actions bot added the Priority: 1 Do this week, receiving emails, time sensitive, . [managed] label Aug 14, 2025
Copilot

This comment was marked as outdated.

@mekarpeles mekarpeles requested a review from Copilot August 14, 2025 19:52
Copy link
Contributor

Copilot AI left a comment

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 fixes the tag_release functionality by improving cleanup handling and preventing potential issues with directory operations. The changes focus on making the cleanup process more robust and ensuring it happens consistently at the end of all deployment scenarios.

  • Enhanced directory existence checks before cleanup operations
  • Added cleanup for openlibrary directory before cloning to prevent conflicts
  • Centralized cleanup to occur at the end of all deployment paths

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mekarpeles mekarpeles force-pushed the fix-deploy-tag-release branch from b0d71e6 to dcd716b Compare August 22, 2025 14:09
@mekarpeles mekarpeles merged commit 78c079e into master Aug 22, 2025
6 of 7 checks passed
@mekarpeles mekarpeles deleted the fix-deploy-tag-release branch August 22, 2025 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: 1 Do this week, receiving emails, time sensitive, . [managed]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deploy Tag Release fails

2 participants