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 git fetch to Auto Update Submodule #104

Merged
merged 1 commit into from
Sep 15, 2020
Merged

Add git fetch to Auto Update Submodule #104

merged 1 commit into from
Sep 15, 2020

Conversation

SudharakaP
Copy link
Member

This adds git fetch to sync with the remote before checking out the submodules in jhipster-online sync action.

Related to jhipster/jhipster-online#236

This syncs with the remote by executing git fetch before checkout of the gh-pages branch in the submodule.

Related to Related to jhipster/jhipster-online#236
@deepu105
Copy link
Member

deepu105 commented Sep 14, 2020 via email

@SudharakaP
Copy link
Member Author

@deepu105 : Yeah, there's several different methods mentioned here; https://stackoverflow.com/questions/5989592/git-cannot-checkout-branch-error-pathspec-did-not-match-any-files-kn. Let us try this first and if that doesn't work we can try another one. Would you be able to give me write access to this repo? Then I can try some things out and fix it if you'd like. 🤔

@deepu105
Copy link
Member

@SudharakaP yes sure

@deepu105 deepu105 merged commit b4acc57 into jhipster:gh-pages Sep 15, 2020
@deepu105
Copy link
Member

I added you to the team, let me know if there is any issue

@deepu105
Copy link
Member

also please note that the default branch for this repo have changed. gh-pages should be considered a read-only branch now and is automatically updated by the deploy.sh script in the src branch

@SudharakaP SudharakaP deleted the add-git-fetch-before-checkout-submodule branch September 15, 2020 14:19
@SudharakaP
Copy link
Member Author

@deepu105 : Thanks for giving me write access. Let me try some things as this PR didn't seem to fix it. 👍🏼

@SudharakaP
Copy link
Member Author

@deepu105 : I've fixed it and pushed the changes to both src and gh-pages branches. Hope now it works properly. 😄

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.

None yet

2 participants