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

Unwrap exception to get the bottom-most cause #2

Merged
merged 1 commit into from Feb 15, 2012

Conversation

trptcolin
Copy link
Collaborator

Looks like I made a bone-headed mistake with the last pull request and omitted the unwrapping that I was doing in REPL-y, so things like clojure.core/ wouldn't work. Sorry about that. I made sure this one comes with a test that fails before the fix is in there.

Also add a test and make sure it fails before the fix.
ninjudd added a commit that referenced this pull request Feb 15, 2012
Unwrap exception to get the bottom-most cause
@ninjudd ninjudd merged commit e674c3f into ninjudd:master Feb 15, 2012
@ninjudd
Copy link
Owner

ninjudd commented Feb 15, 2012

released to 0.2.1 to clojars

@trptcolin
Copy link
Collaborator Author

Sweet, thanks.

@ninjudd
Copy link
Owner

ninjudd commented Feb 15, 2012

Sorry it took so long. I lost the email in my inbox.

On Feb 15, 2012, at 11:29 AM, Colin Jones wrote:

Sweet, thanks.


Reply to this email directly or view it on GitHub:
#2 (comment)

@trptcolin
Copy link
Collaborator Author

No worries, this'll make it into reply & lein in plenty of time for the pre-release.

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

Successfully merging this pull request may close these issues.

None yet

2 participants