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 submodule initialization commands to docs #5459

Merged
merged 2 commits into from
May 27, 2022

Conversation

twosdai
Copy link
Contributor

@twosdai twosdai commented Jul 23, 2021

No issues exists around this issue as far as I can tell, it may relate to the work done in #5455 so reasonably it could be added to that PR.

This is a slight addition to the documentation when downloading and initializing git submodules if they are not setup when first pulled there are some commands which can be ran to help the user install them.

specifically git submodule init && git submodule update

Technical

No technical changes, just some documentation updates for getting started.

Testing

Clone the repository using HTTP, run git submodule init and then git submodule update from the root of the repository

The following should appear:

Cloning into '/Users/$user/git/external/openlibrary/vendor/infogami'...
Cloning into '/Users/$user/git/external/openlibrary/vendor/js/wmd'...
Submodule path 'vendor/infogami': checked out '0b05f20b0323a3078fddc8a9cfd4455cc68faa3a'
Submodule path 'vendor/js/wmd': checked out '7d7b5287bed949eee809182c07c4170309ddbf06'

Screenshot

No UI changes for this PR

Stakeholders

@RayBB I see that you have another Docs related ticket, I am not sure who the correct stakeholders are for this as it is my first PR.

@jamesachamp

@RayBB
Copy link
Collaborator

RayBB commented Jul 23, 2021

@twosdai thanks for contributing to the docs. I don't have merge power but looks good to me 👍

@cdrini cdrini self-assigned this Jul 26, 2021
@twosdai
Copy link
Contributor Author

twosdai commented Aug 2, 2021

Let me know if there is anything else I need to do in order to help this merge along, just don't want it to sit around for ever and die of atrophy.

@cdrini

@twosdai
Copy link
Contributor Author

twosdai commented Mar 4, 2022

@cdrini Just looping back to this as I saw it in my open pull requests, if there is someone else I should bother about it LMK.

@cdrini cdrini force-pushed the DockerReadmeUpdate branch from 95a59d7 to 1632e62 Compare May 27, 2022 22:35
Copy link
Collaborator

@cdrini cdrini left a comment

Choose a reason for hiding this comment

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

My apologies this slipped through the cracks! Lgtm 👍

@cdrini cdrini changed the title update for submodule initialization Add submodule initialization commands to docs May 27, 2022
@cdrini cdrini merged commit 3cc4d18 into internetarchive:master May 27, 2022
cclauss pushed a commit to cclauss/openlibrary that referenced this pull request May 31, 2022
Co-authored-by: Daniel Wasserlauf <Daniel.Wasserlauf@marklogic.com>
Co-authored-by: Drini Cami <cdrini@gmail.com>
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