Jordan Petridis
b850a658ed
pre-commit: Update the indent hook to work with our fork
...
We now have gst-indent-1.0 [1] which is a stripped down
fork of the version we expect. It's also using meson,
which is a bonus.
[1] https://gitlab.freedesktop.org/gstreamer/gst-indent
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/6624 >
2024-05-28 13:27:57 +00:00
Jordan Petridis
d72d79d85a
pre-commit: Avoid prefixing string comparissons with x
...
This was working around bugs in various shells, but this
problem has been fixed for a decade now.
https://www.shellcheck.net/wiki/SC2268
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/6652 >
2024-04-24 12:37:31 +00:00
Jordan Petridis
490deafcbe
git-hooks/pre-commit-python.hook: Fix typos
...
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/6652 >
2024-04-24 12:37:31 +00:00
Jordan Petridis
a12881e2e4
git-hooks/pre-commit-python.hook: Specify encoding for the open call
...
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/6652 >
2024-04-24 12:37:31 +00:00
Jordan Petridis
7b81d081ad
pre-commit: Stop using legacy backticks in shell
...
We can repalce them with the simpler $(foo) syntax instead
https://www.shellcheck.net/wiki/SC2006
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/6652 >
2024-04-24 12:37:31 +00:00
Tim-Philipp Müller
da3229d0cc
git-hooks: use bash for pre-commit-invoke-python
...
Since it contains bashisms.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/5091 >
2023-07-31 17:55:57 +00:00
Nirbheek Chauhan
fcf590a86e
git-hooks: Not all OSes ship Python 3 under "python"
...
macOS doesn't, for instance. Use a shell script to detect which python
to run, which is always available on all OSes due to how git is
packaged on Windows.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/5060 >
2023-07-20 11:00:35 +00:00
L. E. Segovia
3dc1b12f13
git-hooks: Allow running the precommit hook in Windows
...
Windows based Python does not install a python3 executable, that one is
managed by the Microsoft Store as an alias exclusively.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/4968 >
2023-07-05 16:04:07 +00:00
Colin Kinloch
39f15af038
git: Show all suggestions on pre-commit hook
...
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3506 >
2022-12-02 17:53:34 +00:00
Andoni Morales Alastruey
7604c7b088
csharp: check format in the pre-commit hook
...
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/983 >
2022-04-05 08:28:50 +00:00
Thibault Saunier
57bfb85bc5
hooks:python: Max line length should be 120
...
As we ignore line length in pycodestyle and we have many places with
line with more than 80 and less than 120 chars
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1121 >
2021-10-14 12:59:29 +00:00
Nicolas Dufresne
f54c2a2bac
move_mrs: Disable our commit hook during the move
...
As GNU indent version can be different on the user system, we see some
differences during migration thas causes conflicts. Making cherry-pick
difficults to recover without breaking the style temporily. Note that
cherry-pick continuation does not allow passing the -n option to skip
the hooks.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1017 >
2021-10-04 12:56:47 +00:00
Nicolas Dufresne
3037fde5eb
Move commit gst-indent hook to the root
...
This renable at meson setup time the installation of the gst-indent
commit hook. The hooks were kept from gst-devtools as this set supports
both C checks and Python checks.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/904 >
2021-09-24 17:47:01 -03:00
