jhipster / jdl-studio

JDL Studio is an online JHipster Domain Language visual editor
https://start.jhipster.tech/jdl-studio/
Apache License 2.0
405 stars 114 forks source link

Add git fetch to Auto Update Submodule #104

Closed SudharakaP closed 3 years ago

SudharakaP commented 3 years ago

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

Related to https://github.com/jhipster/jhipster-online/pull/236

deepu105 commented 3 years ago

Hmmm weird, I tried this but didn't work. May be I did something wrong. I'll check again tomorrow

On Mon, 14 Sep 2020, 8:02 pm Sudharaka Palamakumbura, < notifications@github.com> wrote:

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 https://github.com/jhipster/jhipster-online/pull/236

You can view, comment on, or merge this pull request online at:

https://github.com/jhipster/jdl-studio/pull/104 Commit Summary

  • fix: Add git fetch before checkout of the gh-pages branch

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jhipster/jdl-studio/pull/104, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIOKFYCUAZRBF5BWGQKELLSFZLFBANCNFSM4RL4VQ3A .

SudharakaP commented 3 years ago

@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 commented 3 years ago

@SudharakaP yes sure

deepu105 commented 3 years ago

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

deepu105 commented 3 years ago

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 commented 3 years ago

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

SudharakaP commented 3 years ago

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