readthedocs.org
readthedocs.org copied to clipboard
VCS: remove unused methods to "update" an existing copy
Closes #8940 Related #9108 Related #9367
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
There were some changes in our code, but this PR is still useful. I'll try to come back to it when I have some extra time.
I updated this PR but there are still some extra work to do, so I'm leaving it again 😄
IMO, it would be good to deprecate and remove some VCS as already proposed (mainly SVN and bzr) to reduce the complexity and scope of this refactor.
Also, we should probably avoid keep using gitpython
and go back to parse git's output (I think @stsewd already mentioned this in another issue). However, this should be simpler now because we are already doing this with ls-remote
and it's working fine. Branches and tags will use the same command to get these. The only new command we will be introducing to parse its output would be git submodule
to discover them.
I updated this PR again. The main new work here is that it makes the new Git pattern the default. I has been enabled for more than a week already and we haven't had any support issue about this. So, I think we are ready to remove the feature flag from our code.
IMO, it would be good to deprecate and remove some VCS as already proposed (mainly SVN and bzr) to reduce the complexity and scope of this refactor.
I'm still 👍🏼 on this, but I'm not going to cover it on this PR since it requires to follow our deprecation plan. I will do this work on #8840 in the next sprint.
Also, we should probably avoid keep using
gitpython
and go back to parse git's output
We are tracking this in another issue, so we will be following it there: https://github.com/readthedocs/readthedocs-corporate/issues/1524
I will update this PR once Santos merges the work from https://github.com/readthedocs/readthedocs.org/pull/10594 so we don't compete for this chunk of code.
Blocked by https://github.com/readthedocs/readthedocs.org/pull/10606
This is ready for review now 👍🏼
Is it possible to tell a build to use the default prior to this change? It seems possible that this change broke dd-trace-py's release notes workflow. https://github.com/DataDog/dd-trace-py/issues/7176
@emmettbutler it's not possible, if you need to unshallow you can follow the example from https://docs.readthedocs.io/en/stable/build-customization.html#unshallow-git-clone, or maybe the suggestion from https://github.com/readthedocs/readthedocs.org/pull/10795/.