Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Conversation

eogas
Copy link

@eogas eogas commented Sep 30, 2016

No description provided.

@eogas
Copy link
Author

eogas commented Sep 30, 2016

Or should the .gitmodules file use the public URL for this repo?

@shana
Copy link
Contributor

shana commented Oct 3, 2016

@eogas Ooops, apologies about breaking the public builds, the repo was private for a bit there. Yes, it should definitely use the public URL.

@eogas
Copy link
Author

eogas commented Oct 3, 2016

Cool, sorry for the bad PR. I figured it was one or the other, but I'm not really familiar with the whole git modules thing.

@eogas eogas closed this Oct 3, 2016
shana added a commit that referenced this pull request Oct 4, 2016
@shana
Copy link
Contributor

shana commented Oct 4, 2016

@eogas No worries, it was a useful PR! We would have missed the problem otherwise. It's fixed in master now, so you should be able to update the submodules now.

@eogas
Copy link
Author

eogas commented Oct 5, 2016

Sounds good. Thanks!

shana added a commit that referenced this pull request Oct 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants