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

Added compatibility for non-MacOS X developers #208

Merged
merged 9 commits into from Oct 16, 2016

Conversation

jwang1919
Copy link
Contributor

@jwang1919 jwang1919 commented Oct 16, 2016

Fixes bugs involved with developing this project on Windows. These fixes should also apply for Linux developers but can't confirm it.

Bugs Fixed:
npm install will fail when attempting to install fsevents dependency
See: npm/npm#14042

npm test will fail because of CRLF linebreaks
Bug looks like this: sindresorhus/matcher#4
Fixed using .gitattributes: https://help.github.com/articles/dealing-with-line-endings/

Also added *.idea for IntelliJ developers =)

Copy link
Contributor

@outoftime outoftime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sick! Nice! Just a couple things then happy to merge : )

@@ -0,0 +1,6 @@
* text eol=lf
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool!

@@ -2,3 +2,4 @@ static/compiled
node_modules
npm-debug.log
bower_components
*.idea
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it’s best to use a global excludes file to keep out artifacts generated by the editor of your choice—hence no .swp etc. in here even though I’m a vim user. Would that work for you?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure can do.

@@ -130,6 +130,7 @@ for all the sordid details.
* [Alejandro AR](https://github.com/kinduff)
* [Vaibhav Verma](https://github.com/v)
* [Alex Pelan](https://github.com/alexpelan)
* [Jesse Wang](https://github.com/jwang1919)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Be sure to add yourself to the package.json as well!

Copy link
Contributor Author

@jwang1919 jwang1919 Oct 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh didn't notice* another contributors section!

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