-
Notifications
You must be signed in to change notification settings - Fork 280
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
Examples cleanup #196
Examples cleanup #196
Conversation
Signed-off-by: Robert Vojta <rvojta@me.com>
Signed-off-by: Robert Vojta <rvojta@me.com>
Signed-off-by: Robert Vojta <rvojta@me.com>
Signed-off-by: Robert Vojta <rvojta@me.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contributions!
@@ -31,6 +31,10 @@ members = [ | |||
"crossterm_screen" | |||
] | |||
|
|||
exclude = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, altough is that a requirement, because I believe it is excluded by default right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's excluded in a way you can't do stuff like cargo run --example snake
. But w/o this section, you can't run snake via cargo run
even if you're in the snake folder.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
right, I didn't know of that option, I was looking for that behavior, previously I would just put it in the 'member' tags, run it, and remove it, but that feels hacky and it is. Thanks for pointing out the alternative.
examples/program_examples/command_bar.rs
toexamples/command_bar.rs
cargo run --example command_bar
examples/program_examples
from the workspacesnake
&first_depth_search
, see below