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

Remove vendored roles from Repository #32

Merged
merged 3 commits into from Jan 26, 2015

Conversation

Projects
None yet
3 participants
@leonelgalan
Contributor

leonelgalan commented Jan 23, 2015

We should use the ansible-galaxy to retrieve vendor roles. That way we don't have to keep their code on our repo. Maybe during installation, we can ansible-galaxy install zzet.rbenv and jnv.unattended-upgrades

@leonelgalan leonelgalan self-assigned this Jan 23, 2015

@leonelgalan

This comment has been minimized.

Contributor

leonelgalan commented Jan 23, 2015

⚠️ Remember to do ansible-galaxy install -r requirements.yml --force once you pull down this commits. ⚠️

@leonelgalan leonelgalan added ready and removed in progress labels Jan 23, 2015

@iandonovan

This comment has been minimized.

Contributor

iandonovan commented Jan 23, 2015

I like it, but I'd wait for input to come in that isn't mine before merging. I'm not too familiar with Tape's architecture. I say :shipit: though.

@leonelgalan

This comment has been minimized.

Contributor

leonelgalan commented Jan 23, 2015

JK, we are not using laggyluke.nodejs after all.

BrandonMathis added a commit that referenced this pull request Jan 26, 2015

Merge pull request #32 from smashingboxes/feature/remove_vendor_roles…
…_from_repo

Remove vendored roles from Repository

@BrandonMathis BrandonMathis merged commit ca2748c into master Jan 26, 2015

@BrandonMathis BrandonMathis removed the ready label Jan 26, 2015

@leonelgalan leonelgalan deleted the feature/remove_vendor_roles_from_repo branch Feb 4, 2015

@leonelgalan leonelgalan removed their assignment Mar 13, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment