Skip to content

Make Windows-friendly#31

Merged
dakrone merged 2 commits intodakrone:masterfrom
crinklywrappr:master
Jan 26, 2017
Merged

Make Windows-friendly#31
dakrone merged 2 commits intodakrone:masterfrom
crinklywrappr:master

Conversation

@crinklywrappr
Copy link
Copy Markdown
Contributor

lein-bikeshed does not run on Windows, but with these small tweaks it does. Bonus: removes the cli dep.

@crinklywrappr
Copy link
Copy Markdown
Contributor Author

If this is merged then I can also deal with the complaint for pull request #23 ... it's just a simple filter on the files using http://docs.oracle.com/javase/7/docs/api/java/nio/file/FileSystem.html#getPathMatcher(java.lang.String)

@dakrone
Copy link
Copy Markdown
Owner

dakrone commented Jan 22, 2017

Thanks for opening this!

@DoubleAgent I assume you have a windows machine that this works on? I don't have any windows machines to test on unfortunately

@crinklywrappr
Copy link
Copy Markdown
Contributor Author

crinklywrappr commented Jan 22, 2017

I ran against lein-bikeshed on Windows 7 & 10.

@crinklywrappr
Copy link
Copy Markdown
Contributor Author

Hm. Solves Issue #4

@dakrone dakrone merged commit 7d32902 into dakrone:master Jan 26, 2017
@dakrone
Copy link
Copy Markdown
Owner

dakrone commented Jan 26, 2017

Merged, thanks @DoubleAgent !

immoh added a commit to immoh/lein-bikeshed that referenced this pull request Jan 15, 2018
This was accidentally removed in dakrone#31.
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