Skip to content

Fixed issues https://github.com/snoe/node-cljfmt/issues/7#8

Open
oneness wants to merge 4 commits into
snoe:masterfrom
oneness:master
Open

Fixed issues https://github.com/snoe/node-cljfmt/issues/7#8
oneness wants to merge 4 commits into
snoe:masterfrom
oneness:master

Conversation

@oneness

@oneness oneness commented Aug 20, 2019

Copy link
Copy Markdown

No description provided.

@snoe

snoe commented Aug 22, 2019

Copy link
Copy Markdown
Owner

Thanks @oneness , can you please update the patch to only add the relevant line without the whitespace changes. Also please remove the binary as I'll compile that myself when I release.

@oneness

oneness commented Aug 22, 2019

Copy link
Copy Markdown
Author

Just pushed as you request. Please merge if ok.

@snoe

snoe commented Aug 23, 2019

Copy link
Copy Markdown
Owner

@oneness Sorry for my imprecise language, remove the /bin/cljfmt line from gitignore, and in your patch reset to the version of the binary on master. I want the binary to be in the repo, but I can't accept patches to the binary itself.

@oneness

oneness commented Aug 26, 2019

Copy link
Copy Markdown
Author

@snoe Just pushed the .gitignore.

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