Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prep patch release 8.3.1 #2619

Merged
merged 12 commits into from
Jul 22, 2024
Merged

Prep patch release 8.3.1 #2619

merged 12 commits into from
Jul 22, 2024

Conversation

vmcj
Copy link
Member

@vmcj vmcj commented Jul 20, 2024

We need this to release the docker containers for 8.3 (or we need to fix the script there).

I forgot to use the -x flag for some of the commits but as those are in order it's not that hard to find the original commits if needed.

vmcj and others added 10 commits July 20, 2024 10:15
We now track the autoload_runtime file and create it as
the normal user and because it exists we don't recreate it in the
install-domserver step as root (assuming it runs with `sudo`). It will
still create it if it doesn't exist, because domserver is a dependency for
install-domserver (and in that case `composer` would run as root again,
so prompting the user to accept this risk).

This also works for the inplace targets as those depend on `build`
which runs the `domserver` part.

(cherry picked from commit b404383)
This triggered an error and broke the webstandard test.

(cherry picked from commit 6862840)
The authentication fails so needs to be rewritten with the new github
action.
It seems Python complains about this for some Python versions.
This reverts commit 0219d8e.

(cherry picked from commit 98c29f7)
@vmcj vmcj force-pushed the prep_patch_release_8.3.1 branch from 8f522ff to 4f104e7 Compare July 20, 2024 08:54
vmcj added 2 commits July 20, 2024 11:54
In main this file already uses another image so composer was missing.
@vmcj vmcj force-pushed the prep_patch_release_8.3.1 branch from c569e75 to 8160e4b Compare July 20, 2024 09:54
@vmcj vmcj requested a review from a team July 20, 2024 10:10
@vmcj vmcj merged commit 92e3ad7 into DOMjudge:8.3 Jul 22, 2024
22 checks passed
@vmcj vmcj deleted the prep_patch_release_8.3.1 branch July 22, 2024 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants