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
Undefined in requirejs #34
Comments
@wenerme I'm having the same problem, were you able to figure this out? |
I figured it out. The AMD definition is incorrect at log.coffee:110:
should be
@adamschwartz I can make a pull request with this fix if desired, or you can do it since it's so small. |
Thanks @wenerme & @dav- for reporting this. Interestingly enough, it looks like this was fixed with #32 in the JS, but not the original Coffeescript. Are we sure that this is now correct? Let’s also loop in @alessioalex, the original author of the AMD stuff (#30), and @rodneyrehm, the author of the JS fix in (#32), for some extra eyes. |
@adamschwartz I found that just before you commented. It's interesting that it's fixed in the JS, because the package I got from Bower seemed to have the unfixed JS. |
#32 only fixed the compile target, not the compiled source. so any time someone compiled the CoffeeScript, the error would reappear. I have no Idea how I only fixed the JS, but not the CS. Shame on me :/ |
Np @rodneyrehm :) Alllright. 86a79c7 and fadf243 should bring us home. Thx all! |
main.js
bower.json
.bowerrc
The text was updated successfully, but these errors were encountered: