Skip to content

feat: conclude manifest-sync experiment - #624

Open
srtaalej wants to merge 1 commit into
mainfrom
release-manifest-sync
Open

feat: conclude manifest-sync experiment#624
srtaalej wants to merge 1 commit into
mainfrom
release-manifest-sync

Conversation

@srtaalej

@srtaalej srtaalej commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Removes the manifest-sync experiment flag and enables two-way manifest sync by default
  • Unhides slack manifest sync command
  • Replaces the old confirm prompt flow in shouldUpdateManifest with a call to manifest.Sync
  • Updates experiment docs

Test plan

  • make lint passes
  • make test passes
  • Manual test: slack manifest sync works without --experiment manifest-sync
  • Manual test: slack deploy triggers sync on manifest mismatch

@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.23%. Comparing base (9582a26) to head (156617e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #624      +/-   ##
==========================================
- Coverage   72.28%   72.23%   -0.05%     
==========================================
  Files         238      238              
  Lines       20087    20036      -51     
==========================================
- Hits        14519    14474      -45     
+ Misses       4299     4294       -5     
+ Partials     1269     1268       -1     

☔ 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.

@srtaalej srtaalej self-assigned this Aug 5, 2026
@srtaalej srtaalej added enhancement M-T: A feature request for new functionality experiment Experimental feature accessed behind the --experiment flag or toggle semver:minor Use on pull requests to describe the release version increment labels Aug 5, 2026
@srtaalej srtaalej added this to the Next Release milestone Aug 5, 2026
@srtaalej
srtaalej marked this pull request as ready for review August 5, 2026 20:10
@srtaalej
srtaalej requested review from a team as code owners August 5, 2026 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement M-T: A feature request for new functionality experiment Experimental feature accessed behind the --experiment flag or toggle semver:minor Use on pull requests to describe the release version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant