Skip to content

fix(build): order release preparation tasks#6219

Merged
markphelps merged 1 commit into
v2from
fix/release-prep-order
Jul 18, 2026
Merged

fix(build): order release preparation tasks#6219
markphelps merged 1 commit into
v2from
fix/release-prep-order

Conversation

@markphelps

Copy link
Copy Markdown
Collaborator

Summary

Prevent clean from deleting UI assets after the Vite build during release preparation.

  • Run clean before ui:build in mise run prep
  • Eliminate the race that caused GoReleaser to fail resolving ui/embed.go assets

Validation

  • mise tasks validate
  • mise run prep
  • go build -tags=assets ./cmd/flipt
  • Fresh code review (no findings)

mise run test is blocked locally by the inherited commit.gpgSign configuration in Git fixture tests.

@markphelps
markphelps requested a review from a team as a code owner July 18, 2026 00:44
@markphelps markphelps added the v2 Flipt v2 label Jul 18, 2026
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 18, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verdict: approve

Fixes a real race condition in the release prep task by ensuring clean completes before ui:build runs. Looks correct — no changes requested.

🤖 Automated review by the Flipt PR review agent.

@codecov

codecov Bot commented Jul 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.27%. Comparing base (1a4ee81) to head (f50dece).

Additional details and impacted files
@@           Coverage Diff           @@
##               v2    #6219   +/-   ##
=======================================
  Coverage   62.27%   62.27%           
=======================================
  Files         145      145           
  Lines       14815    14815           
=======================================
  Hits         9226     9226           
  Misses       4829     4829           
  Partials      760      760           
Flag Coverage Δ
integrationtests 33.50% <ø> (ø)
unittests 53.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@markphelps
markphelps merged commit 2ba7835 into v2 Jul 18, 2026
36 checks passed
@markphelps
markphelps deleted the fix/release-prep-order branch July 18, 2026 01:10
@github-project-automation github-project-automation Bot moved this to Done in Flipt V2 Jul 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files. v2 Flipt v2

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant