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

Remove submodules now that the new CI doesn't need them. #3131

Merged
merged 1 commit into from
Feb 18, 2020

Conversation

nat-henderson
Copy link
Contributor

Release Note Template for Downstream PRs (will be copied)


Copy link
Contributor

@danawillow danawillow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 🎉 🎉

nice branch name!

@rambleraptor, does this mess up your workflow at all?

@rambleraptor
Copy link
Contributor

burn them with fire please!

@rambleraptor
Copy link
Contributor

I keep a separate folder for each MM branch (and I'm terrible with deleting them when I'm done). Thanks to those submodules, I have 50GB of copies of MM/Ansible/Terraform lying around on my hard drive right now...

@nat-henderson
Copy link
Contributor Author

Cool, shutting them down now, then.

@nat-henderson nat-henderson merged commit e9b3287 into GoogleCloudPlatform:master Feb 18, 2020
nathkn pushed a commit to nathkn/magic-modules that referenced this pull request May 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants