-
Notifications
You must be signed in to change notification settings - Fork 745
Mark C++ exceptions as unsupported in the book #1209
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
Conversation
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.
LGTM, we need to figure out the travis CI issues that are also hitting #1207.
Thanks Leo! :)
My pleasure, that's not really a lot 😋 but if it can help! |
@bors-servo r+ |
📌 Commit 3f35a93 has been approved by |
Mark C++ exceptions as unsupported in the book This should close #1172 . It's my first contribution to the book, so I hope it's not too far from expected usage!
💔 Test failed - status-travis |
@bors-servo retry |
Mark C++ exceptions as unsupported in the book This should close #1172 . It's my first contribution to the book, so I hope it's not too far from expected usage!
💔 Test failed - status-travis |
Srsly... |
Thanks a lot! :) |
Heh, sometimes CI isn't all that nice :þ My pleasure :) |
This should close #1172 . It's my first contribution to the book, so I hope it's not too far from expected usage!