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

Remove workspace #4212

Merged
merged 8 commits into from
May 31, 2020
Merged

Remove workspace #4212

merged 8 commits into from
May 31, 2020

Conversation

topecongiro
Copy link
Contributor

This PR removes the workspace and replaces it with a flat crate structure.

My apologies for going back and forth; this change is required because cargo publish won't work with the current project structure. Also, we no longer need not have the workspace because we won't be using different versions of rustfmt-as-a-library to manage stability guarantee.

I have added some features for binary targets so that rustfmt-lib users won't need to pull in dependencies for binaries.

@topecongiro topecongiro merged commit 2d0ea2f into rust-lang:master May 31, 2020
@topecongiro topecongiro deleted the remove-workspace branch May 31, 2020 13:48
@karyon
Copy link
Contributor

karyon commented Oct 28, 2021

backport not needed as this applies to the crate separation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants