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

Put answers in a subrepo #2545

Merged
merged 2 commits into from
Apr 16, 2020
Merged

Conversation

matthewturk
Copy link
Member

This PR puts our answer-store directory in a submodule. Unless someone clones --recursive, by default they won't get the full history of the answer tests.

One open question I have is if I can or should do anything to suggest a depth of the clone; from what I can tell, specifying --depth in the git submodule add command wouldn't change anything for folks cloning the repo, but I may be misreading.

Travis should clone by default all the submodules, so let's see how it goes...

@matthewturk matthewturk added tests: running tests Issues with the test setup infrastructure Related to CI, versioning, websites, organizational issues, etc labels Apr 15, 2020
@matthewturk
Copy link
Member Author

@munkm suggests this go on 4.0 -- I will leave this open and issue a new PR too.

@munkm munkm merged commit f8ff205 into yt-project:master Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Related to CI, versioning, websites, organizational issues, etc tests: running tests Issues with the test setup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants