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

chore(cli): Remove redundant CLI feature flags #3955

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

theduke
Copy link
Contributor

@theduke theduke commented Jun 5, 2023

The CLI previously had a bunch of feature flags, which were only left in
for legacy support, but which do not change behaviour anymore.

This commit removes the feature flags in question.

(flags: webc_runner, cache, wasi, debug, emscripten, cache-blake3-pure )

Closes #3954

@theduke theduke requested a review from syrusakbary as a code owner June 5, 2023 16:31
Copy link
Contributor

@ptitSeb ptitSeb left a comment

Choose a reason for hiding this comment

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

I thought publish.py als needed some changes, but now, it's fine.

@theduke theduke force-pushed the issue-3954-remove-old-cli-feature-flags branch 2 times, most recently from 9386c8f to 795b948 Compare June 5, 2023 16:43
The CLI previously had a bunch of feature flags, which were only left in
for legacy support, but which do not change behaviour anymore.

This commit removes the feature flags in question.

(flags: webc_runner, cache, wasi, debug, emscripten, cache-blake3-pure )

Closes #3954
@theduke theduke force-pushed the issue-3954-remove-old-cli-feature-flags branch from 795b948 to 40c409a Compare June 5, 2023 17:35
@theduke theduke enabled auto-merge June 5, 2023 21:19
@theduke theduke merged commit a7ab342 into master Jun 5, 2023
@theduke theduke deleted the issue-3954-remove-old-cli-feature-flags branch June 5, 2023 21:51
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.

CLI: Remove old deprecated feature flags
3 participants