Fixes #34

Casey authored 1 month ago and Pablo Correa Gomez committed 1 month ago

We are planning to migrate to marge-bot for handling merging commits,
and with it adopt a new way of linking commits to their source MRs.

Using a trailer with Part-of: !MR-ID results in the commits being linked
to the MR, and the GitLab web interface inserting a hyperlink in its
place making it easier to jump to the source MR.

Let's adopt this for MRs we still merge manually now.

Use the full URL to the MR rather than just the ID, this way if we ever
split or merge repos it will still be easily possible to trace commits
back to their original MRs.

Signed-off-by: Caleb Connolly <caleb@postmarketos.org>
Part-of: !70

These are generated by mypyc and we definitely don't want to commit
them.

This is supported by mypyc now, so we can go back to using the shorter
syntax.

This reverts commit 0e1e5986.

Installing this with pipx caused these files to be generated

Otherwise apk won't fetch packages from the systemd extra repo when
using `mrtest upgrade`.

This was merged without maintainer approval and is not a trivial fix, so
it is a violation of our rules for merging. Additionally, I do not agree
with these changes in their current form, see [1] for my reasoning.

This reverts commit da492e4a.

 [1]: pmbootstrap#2534

magdesign authored 3 months ago and Oliver Smith committed 3 months ago

Tweaked-By: Oliver Smith <ollieparanoid@postmarketos.org>

This resolves a cyclic dependency in pipeline, and is also a more
programatic pattern

It makes the most sense to have it there. We move it to mrhlpr since
that's where it was originally defined

Fixes #30

They are not used anywhere else, so let's not expose them to make it
more clear

This will be useful to do some compartmerization of the code later, as
this will become more complex with changes related to Alpine

Pablo Correa Gomez authored 4 months ago and Newbyte committed 4 months ago

There's no need for it, and breaks when trying to merge
postmarketOS/pmaports!5840 (comment 457021)

Fixes #25

Inspired from hexaheximal

Signed-off-by: Robert Eckelmann <longnoserob@postmarketos.org>

Clayton Craft authored 5 months ago and Newbyte committed 4 months ago

This doesn't depend on an auth error to detect if a user has approved a
MR, and fixes #24.

For some reason mr.approval_state().get was 404'ing, causing #24.
[ci:skip-build]: already built successfully in CI

Previously, the string representation for ActionType would be something
like ActionType.OK instead of [OK ] as intended. This resulted in
commands like `$ mrhlpr status` getting an output like this:

    ActionType.OK Changes allowed
    ActionType.OK Clean worktree
    ActionType.NOK Rebase on master
    ActionType.NOK MR-ID in commit msgs
    ActionType.NOK Commit subjects follow format
       668526 doesn't match any regex
    ActionType.NOK Commits are signed

instead of the intended

    [OK ] Changes allowed
    [OK ] Clean worktree
    [NOK] Rebase on master
    [NOK] MR-ID in commit msgs
    [NOK] Commit subjects follow format
       668526 doesn't match any regex
    [NOK] Commits are signed

Fixes 8647dd3b

mypyc is not yet completely stable, and might fail for things that are
correct Python (see
!61 (comment 456440)),
so don't make it a requirement that it passes for merging

Doesn't require to keep in sync all architectures supported by
Alpine, like ppc64le or loongarch64.

magdesign authored 5 months ago and Pablo Correa Gomez committed 5 months ago

Fixes #26

This seems to help with more reliably getting automerge going on the new
GitLab instance … maybe.

We pass this check.

With the parent of this commit, we pass this check.

We already pass this, so might as well enable it.

This makes the values easier to tweak in the future.

And remove equivalent switches from mypy.sh script to avoid
duplication.

Robert Eckelmann authored 8 months ago and Newbyte committed 5 months ago

Co-developed-by: Luca Weiss <luca@lucaweiss.eu>

Robert Eckelmann authored 5 months ago and Luca Weiss committed 5 months ago

In order to make Sphinx be able to list the supported commands, refactor
the argument parsing code to return ArgumentParser object, and align
the function naming closer to how it works in pmbootstrap.

Co-developed-by: Luca Weiss <luca@lucaweiss.eu>

Add some spacing and indent lists correctly.

Pablo Correa Gomez authored 6 months ago and Pablo Correa Gomez committed 5 months ago

GNOME's SSH endpoints areof the form
git@ssh.gitlab.gnome.org:GNOME/Incubator/papers.git and without this
patch, the host could not be found

This also effectively reverts 7d068e20,
because now we actually use the version for something in the program. It
also avoids the problems reported in
#15 to the
best of my knowledge — I tried reproducing the bug as written in there
and did not get any build failures.

The primary purpose of this version command is for making
experimentation with mypyc easier, but I also can imagine it might be
nice for users to be able to get their version of mrtest easily.