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

Revert changing to workspace deps for egui crates #3942

Merged
merged 2 commits into from
Feb 1, 2024

Conversation

emilk
Copy link
Owner

@emilk emilk commented Feb 1, 2024

Workspace dependencies can be annoying.

If you don't set them to default-features=false, then you cannot opt out of their default features anywhere else, and get warnings if you try.

So you set default-features=false, and then you need to manually opt in to the default features everywhere else.
Or, as in my case, forget to do so and break a bunch of things.

I don't have the energy to do this tonight, so I'll just revert.

@emilk emilk merged commit 8f2c866 into master Feb 1, 2024
35 of 36 checks passed
@emilk emilk deleted the emilk/revert-workspace-deps branch February 1, 2024 20:29
@emilk emilk changed the title Emilk/revert workspace deps Revert changing to workspace deps for egui crates Feb 1, 2024
hacknus pushed a commit to hacknus/egui that referenced this pull request Oct 30, 2024
* Closes emilk#3941

Workspace dependencies can be annoying.

If you don't set them to `default-features=false`, then you cannot opt
out of their default features anywhere else, and get warnings if you
try.

So you set `default-features=false`, and then you need to manually opt
in to the default features everywhere else.
Or, as in my case, don't.

I don't have the energy to do this tonight, so I'll just revert.
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.

egui.rs appears to be broken
1 participant