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

Handle Windows filenames in cider-compilation-regex #1384

Merged
merged 2 commits into from
Oct 27, 2015

Conversation

juergenhoetzel
Copy link
Contributor

No description provided.

@bbatsov
Copy link
Member

bbatsov commented Oct 27, 2015

The change looks OK to me. I'd change the second commit message to Match windows file names in cider-compilation-regexp. It'd be nice to mention this in the changelog as well.

@juergenhoetzel
Copy link
Contributor Author

That makes sense: I amended the commit.

@bbatsov
Copy link
Member

bbatsov commented Oct 27, 2015

I still don't see a changelog entry.

The previous regex was too restrictive: Windows file names always
contain a colon.
@juergenhoetzel
Copy link
Contributor Author

😞 Sorry missed out changelog entry in previous commit: Amended again.

bbatsov added a commit that referenced this pull request Oct 27, 2015
Handle Windows  filenames in cider-compilation-regex
@bbatsov bbatsov merged commit 3dcce71 into clojure-emacs:master Oct 27, 2015
@bbatsov
Copy link
Member

bbatsov commented Oct 27, 2015

👍

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.

2 participants