python / python-docs-fr

Mirroir de https://git.afpy.org/AFPy/python-docs-fr
https://docs.python.org/fr/
Other
353 stars 268 forks source link

merge.py: Use git ls-files to get list of downstream PO files #1883

Closed jeanas closed 2 years ago

jeanas commented 2 years ago

The previous method could lead to problems if a dependency installed within venv/ shipped with its own PO files.

jeanas commented 2 years ago

Cf. https://github.com/python/python-docs-fr/pull/1880#issuecomment-1133940739.

JulienPalard commented 2 years ago

Wow TIL git does its own glob expansion by default.

Wow that's funny then in bash, as git and bash don't interpret *.po the same way, and as bash leaves the raw *.po when no matches are to be found:

But there's no bash involved in your PR so it's 100% right, git will expand properly every time, nicely done!

deronnax commented 2 years ago

hat down 👏