Skip to content

fix(build): Ungate ProgressBarMode::Response from managed feature#3247

Merged
runningcode merged 1 commit intomasterfrom
no/fix-managed-progress-bar
Mar 26, 2026
Merged

fix(build): Ungate ProgressBarMode::Response from managed feature#3247
runningcode merged 1 commit intomasterfrom
no/fix-managed-progress-bar

Conversation

@runningcode
Copy link
Copy Markdown
Contributor

@runningcode runningcode commented Mar 26, 2026

Summary

  • ProgressBarMode::Response was gated behind #[cfg(not(feature = "managed"))] because its only consumer (download_with_progress) was also managed-gated
  • The new download_installable_build method (used by build download) references this variant without a managed gate, breaking the Docker build (--features managed)
  • The progress bar has nothing to do with managed mode — remove the feature gate from the variant and simplify the response() method

#skip-changelog

🤖 Generated with Claude Code

The `Response` variant was gated behind `#[cfg(not(feature = "managed"))]`
because its only consumer (`download_with_progress`) was also
managed-gated. The new `download_installable_build` method (used by
`build download`) references this variant without a managed gate,
breaking the Docker build which compiles with `--features managed`.

The progress bar has nothing to do with managed mode, so remove the
feature gate from the variant and simplify the `response()` method.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

Fails
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

### Fixes

- Ungate ProgressBarMode::Response from managed feature ([#3247](https://github.com/getsentry/sentry-cli/pull/3247))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against e431a6b

@runningcode runningcode marked this pull request as ready for review March 26, 2026 14:32
@runningcode runningcode requested review from a team and szokeasaurusrex as code owners March 26, 2026 14:32
@runningcode runningcode merged commit e744afd into master Mar 26, 2026
25 of 26 checks passed
@runningcode runningcode deleted the no/fix-managed-progress-bar branch March 26, 2026 14:36
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