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

Fix #75. Show resolved sources files to avoid issues like #103. Unify coding style #105

Closed
wants to merge 5 commits into from

Conversation

richardgrey
Copy link

Issues #75 fixed by replacing bin file with new one. To avoid problems with line endings in future, end_of_line directive was added to .editorconfig.

@stevenvachon
Copy link

Why was this not put in v0.2 or v0.3? @adam-lynch

@adam-lynch
Copy link
Contributor

I'm not sure why this.emit('log', 'Build app from ' + this.options.files); is needed? Isn't that a different issue? If the API is confusing, then we'll add validation or something. But that's a separate thing to the line-endings problem, right?

@steffenmllr
Copy link
Contributor

@stevenvachon Thanks for you work, could you pls rebase and squash the commit so we can publish this as 0.3.1 and close #75 :)

@adam-lynch
Copy link
Contributor

ping 😄 Getting more reports from other people having the error

@stevenvachon
Copy link

@steffenmllr it's not my PR 😉

@richardgrey please fix this.

@vikbez
Copy link

vikbez commented Jan 29, 2015

👍 please fix this ❤️

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.

5 participants