-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
refactor: exercise evaluation #271
Conversation
pub stderr: String, | ||
} | ||
|
||
impl Drop for FileHandle { |
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.
This is a silly trick to avoid calling clean()
all over the place. I think it's not racier than what we were doing before.
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.
👍
Exercise evaluation (compilation + execution) now uses Results Success/failure messages are standardized
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 job! I'm pushing out a minor release tonight, and I'll land this after if there's nothing more from your side.
pub stderr: String, | ||
} | ||
|
||
impl Drop for FileHandle { |
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.
👍
Awesome. I do have the |
@bors: r+ |
📌 Commit 43dc311 has been approved by |
☀️ Test successful - checks-travis |
refactor: exercise evaluation After working a bit on rust-lang#270, I realized that it'd be useful to first perform a minor refactor of exercise evaluation. * Now we have standard methods to compile + execute that return `Result`s. * Success/failure messages are standardized.
refactor: exercise evaluation After working a bit on rust-lang#270, I realized that it'd be useful to first perform a minor refactor of exercise evaluation. * Now we have standard methods to compile + execute that return `Result`s. * Success/failure messages are standardized.
refactor: exercise evaluation After working a bit on rust-lang#270, I realized that it'd be useful to first perform a minor refactor of exercise evaluation. * Now we have standard methods to compile + execute that return `Result`s. * Success/failure messages are standardized.
After working a bit on #270, I realized that it'd be useful to first perform a minor refactor of exercise evaluation.
Result
s.