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

add component.json, and get rid of dev deps that are private #174

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

xcoderzach
Copy link
Contributor

Starting a pull, don't merge it until you guys make xcoderzach/path an amicus repo.

Also, I got rid of test-things and awesome-docs as deps, since they aren't open source.

EDIT:

they aren't open source.

yet O_o

@amicus-jenkins
Copy link

Can one of the admins verify this patch?

@xcoderzach
Copy link
Contributor Author

So, I fixd all the cyclic dep errors, but didn't actually get rid of the cycles :-( . I think that they're probably indicative of a larger problem. I think that the real solution is to stop having the EndDash monolith object, and instead dependency inject things like setCustomViewGetWhateverMyNameIsFunction

@tobowers
Copy link
Contributor

tobowers commented Jan 7, 2014

@xcoderzach this looks ok to merge to me now. Is that true?

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