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

Don't use browser alias for component-each dependency #17

Merged
merged 1 commit into from
Mar 15, 2016

Conversation

ockham
Copy link
Contributor

@ockham ockham commented Mar 15, 2016

This way, component-css won't break node code/tests when imported.

@ockham
Copy link
Contributor Author

ockham commented Mar 15, 2016

Please point-release after merging!

@ockham ockham force-pushed the dont-user-indexof-browser-alias branch from 2f5fb2e to 052d6db Compare March 15, 2016 17:27
ockham added a commit that referenced this pull request Mar 15, 2016
Don't use browser alias for component-each dependency
@ockham ockham merged commit 414eab0 into component:master Mar 15, 2016
@ockham ockham deleted the dont-user-indexof-browser-alias branch March 15, 2016 17:36
@stephenmathieson
Copy link
Contributor

this broke component support

@ockham
Copy link
Contributor Author

ockham commented Mar 15, 2016

Ugh, sorry about that. Apparently I'm somewhat ignorant about the actual component side of things -- what did I do wrong? (I tried to stick to the pattern of component/classes#27.)

@stephenmathieson
Copy link
Contributor

oh whoops. looks like that broke component support too haha

basically the module name needs to match the repo name, as component loads modules from github (not from npm)

@ockham
Copy link
Contributor Author

ockham commented Mar 15, 2016

Yikes. So the only way to do this is to try...catch require both variants after all, I guess?

@ockham
Copy link
Contributor Author

ockham commented Mar 31, 2016

Fix in #18.

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