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

FIX: Sort out the sync submodule issue 🤔 #1365

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

daquinteroflex
Copy link
Collaborator

After many closed branches attempts including with git subtree

@daquinteroflex daquinteroflex changed the title 🤔 sort out the sync submodule issue FIX: Sort out the sync submodule issue 🤔 Jan 9, 2024
@daquinteroflex
Copy link
Collaborator Author

daquinteroflex commented Jan 9, 2024

So I think this kind-of sorts it:

This PR seems to be building alright
https://readthedocs.com/projects/flexcompute-tidy3ddocumentation/builds/1938919/

PR docs hosted here:
https://docs.flexcompute.com/projects/tidy3d/en/dario-fix_sync_with_submodule/?next=https%3A%2F%2Fdocs.flexcompute.com%2Fprojects%2Ftidy3d%2Fen%2Fdario-fix_sync_with_submodule%2F&ticket=ST-1704812510-leWqwdS0RktzZH1NOxwYb2daYqfUXhpl&subproject=tidy3d

I think it should have checked out the right submodule:

Submodule 'docs/notebooks' (git@github.com:flexcompute/tidy3d-notebooks.git) registered for path 'docs/notebooks'
  Cloning into '/home/runner/work/tidy3d/tidy3d/docs/notebooks'...
  From https://github.com/flexcompute/tidy3d-notebooks
   * branch            dfc784c8a2775c6dda3b9dd4c0169a85a523c2ad -> FETCH_HEAD
  Submodule path 'docs/notebooks': checked out 'dfc784c8a2775c6dda3b9dd4c0169a85a523c2ad'

I am still verifying that the exact file in the documentation is the right output file.

Why "kind of" sorted? Really because this is a patch. It works fine for a release and our docs build fine. There is a bug on the create branch action which leaves other old files that shouldn't interact with the main docs build but it's important to know they're there. I've raised the issue that it doesn't check out the branch properly with old submodules there GuillaumeFalourd/create-other-repo-branch-action#10 . If we find it valuable I can sit down and fix the action for our intentions.

@daquinteroflex daquinteroflex marked this pull request as ready for review January 9, 2024 15:02
@momchil-flex momchil-flex merged commit f8b65db into pre/2.6 Jan 9, 2024
19 checks passed
@momchil-flex momchil-flex deleted the dario/fix_sync_with_submodule branch January 9, 2024 18:00
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.

2 participants