py3-gitlab is now in community:
https://gitlab.alpinelinux.org/alpine/aports/-/commit/b470cf0780124804e17ce7510418b77640a42b0b

[ci:skip-build]: already built successfully in CI

Ruff will recursively lint all files in subdirectories if no list of
files is omitted. This also lets us remove the ShellCheck ignore as the
affected code no longer exist.

[ci:skip-build]: already built successfully in CI

If the list of files is omitted, Ruff will check all files recursively.

Use --diff to get a diff of what's not complicit instead of applying
the changes directly.

This does not cover the entirety of mrhlpr's codebase, and does not even
attempt to start typing mrtest. The types were generated with
MonkeyType, which seems to have worked well to get a starting point for
typing. However, it still required significant manual adjustment and as
such time investment due to some of the type hints being wrong
(presumably since MonkeyType runs off of runtime information and I
didn't manage to trigger every possible code path), but primarily due to
the codebase needing other fixes to make mypy happy.

Overall, I think MonkeyType saved me enough time to be worthwhile using
for future type expansion, but the time investment required is still
significant.

This also introduces the MergeRequestStatus and GitLabOrigin dataclasses
to avoid unnecessary type assertions and improve ease of reading the
code.

Nikita Travkin authored 1 year ago and Pablo Correa Gomez committed 1 year ago

mrtest installs packages from apk files which pins the package version
to the hash of the package. This means that apk would be reluctant to
update packages on which .mrtest- virtual package depends.

This can cause problems to the users who may forget that they have some
packages installed using mrtest, espacially if those were boot-critical
packages.

Add a script that would implement an apk hook that would nag the user
about installed mrtest packages. This hook should be installed in
/etc/apk/commit_hooks.d/ and will be run by apk on every execution.

Fixes: #13

I forgot to do this when migrating from Flake8 to Ruff.

Fixes b09df9bd

flake8 was dropped in b09df9bd, and
this is a left-over that allows us to drop the setup.cfg file

Only 3.8 is not EOL, and it's just some months. Even Debian old stable
is currenly on 3.9

According to
https://wheel.readthedocs.io/en/stable/user_guide.html#building-wheels
universal = 1 is opt-in

See similar patch in pmbootstrap for rationale:
https://gitlab.com/postmarketOS/pmbootstrap/-/commit/27618d5ffdb0b6cbf9471dc01a2ed14e1655ced0

We use this in pmbootstrap, and it is currently necessary to use Ruff.
Issue about moving Ruff to community:
https://gitlab.alpinelinux.org/alpine/aports/-/issues/15693

So we don't need to repeat it.

Repurposing argcomplete as a boolean telling whether the module
argcomplete was found results in a variable with two distinct purposes
and types, which is bad practice and confuses Mypy. As such, don't
assign False to argcomplete if the module is not found and instead just
check for whether it exists in sys.modules to determine whether it was
imported successfully.

This works in practice, but Mypy does not understand it, and I'm not
sure how to best resolve it.

`mrhlpr status 123` on a non-checked-out branch would panic because
`commits_follow_format` was an undefined variable.

Right now you can't switch branch if there are any untracked files. This
is not something that git checkout has a problem with generally, so
don't consider the worktree unclean if the only "impurity" is untracked
files.

Fix the part that inserts an empty line after the first line, before
adding the label.

Add basic logic to detect when we should insert a trailer line at the
end of the commit message, before adding the CI label.

Add a new command-line argument to add the frequently used skip-build
label with the default text.

fixes #11

per the documentation in
https://setuptools.pypa.io/en/latest/userguide/datafiles.html
include_package_data requires also specifying what it is

Forgot to bump the version in mrhlpr/__init__.py, so let's tag a fixup
release.