Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix cleanup in build-windows-modules.sh for release builds #7489

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

dlon
Copy link
Member

@dlon dlon commented Jan 21, 2025


This change is Reviewable

@dlon dlon force-pushed the fix-win-modules-release-clean branch from 230d87f to a7d560e Compare January 21, 2025 08:42
Copy link
Contributor

@MarkusPettersson98 MarkusPettersson98 left a comment

Choose a reason for hiding this comment

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

🪟 🔨
:lgtm:

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@dlon dlon force-pushed the fix-win-modules-release-clean branch from a7d560e to 683aefa Compare January 21, 2025 08:52
@dlon dlon changed the base branch from prepare-2025.3 to main January 21, 2025 08:52
@dlon dlon dismissed MarkusPettersson98’s stale review January 21, 2025 08:52

The base branch was changed.

Copy link
Contributor

@MarkusPettersson98 MarkusPettersson98 left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Copy link
Contributor

@MarkusPettersson98 MarkusPettersson98 left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@dlon dlon force-pushed the fix-win-modules-release-clean branch from 8f8d013 to 3068d8f Compare January 22, 2025 09:08
@dlon dlon merged commit e515f67 into main Jan 22, 2025
33 of 34 checks passed
@dlon dlon deleted the fix-win-modules-release-clean branch January 22, 2025 09:10
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.

2 participants