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

allow coffeescript files along with javascript #69

Closed
wants to merge 3 commits into from

Conversation

mkoryak
Copy link

@mkoryak mkoryak commented Jun 8, 2015

Sorry if this is not ideal, but this PR contains PR #68 because I was lazy :P

Its useful to also allow coffeescript to be required.

clarete and others added 3 commits June 7, 2015 04:54
Debowerify doesn't really help if the bower package doesn't declare the
right files in the `main` section of their bower.json file. This patch
introduces the `overrides` section that allows the user declaring
dependencies to override the main section of the bower.json file of the
package they depend on.

This strategy was copied from `main-bower-files`:

  https://github.com/ck86/main-bower-files#options
@mkoryak
Copy link
Author

mkoryak commented Jun 8, 2015

Bah, this is a really messy way to do this. ill open a new one. Also, your code chunks[node.range[1] + (i + 1)] = st did not work for me, i had to revert to the code in the last release there.

@mkoryak mkoryak closed this Jun 8, 2015
@lattwood
Copy link

@mkoryak any updates?

@mkoryak
Copy link
Author

mkoryak commented Oct 15, 2015

no, i have abandoned debowerify for a better solution. if you want to support coffee files the only change that is requires is /\.(js|coffee)$/.test(file); bit

@lattwood
Copy link

Well, I was hoping for 8b1d523 tbh.

What ended up being your better solution?

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.

3 participants