Skip to content

Update GitHub actions to use Ubuntu 24.04 #23097

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

Merged
merged 7 commits into from
Mar 7, 2025
Merged

Conversation

sgiehl
Copy link
Member

@sgiehl sgiehl commented Mar 4, 2025

Description:

Due to a different font rendering on Ubuntu 24.04 almost all screenshots had to be updated.
I manually check all of them twice and split them into three commits:

18080cb contains all images where changes were only caused by font rendering

f2c1857 contains images were additional changes are visible, as the new font seems to be able to correctly render some additional UTF8 chars

8afa75b contains images with changes in the wording of an error message that is returned by curl

Review

@sgiehl sgiehl force-pushed the actions-ubuntu-24.04 branch from 5c06589 to dcf1798 Compare March 4, 2025 21:57
@sgiehl sgiehl force-pushed the actions-ubuntu-24.04 branch from c7b6939 to a0557b2 Compare March 6, 2025 14:38
@sgiehl sgiehl requested a review from a team March 6, 2025 16:08
@sgiehl sgiehl marked this pull request as ready for review March 6, 2025 16:08
@caddoo caddoo self-assigned this Mar 6, 2025
@sgiehl sgiehl merged commit 5985df9 into 5.x-dev Mar 7, 2025
26 of 27 checks passed
@sgiehl sgiehl deleted the actions-ubuntu-24.04 branch March 7, 2025 10:30
caddoo pushed a commit that referenced this pull request May 1, 2025
* Cherry pick ubuntu-24.04 update excl. submodule update (#23097)

* Update submodules

* Update expected test results
caddoo added a commit that referenced this pull request May 2, 2025
* Cherry pick ubuntu-24.04 update excl. submodule update (#23097)

* Update expected test results

* Ensure login username is provided as string value (#23117)

* Prevent automated linking in emails (#23248)

* Prevent automated linking in emails

* prevent linking on some more elements

* add some tests

---------

Co-authored-by: caddoo <1169490+caddoo@users.noreply.github.com>

* Improve handling of prefixUrl parameter in API.listAllMethods (#23247)

* Fix escaping in realtime report (#23245)

* Ensure all translations required for password confirmation are always loaded (#23176)

* Suppress errors when trying to access javaEnabled property (#23161)

Co-authored-by: caddoo <matthew.caddoo@gmail.com>

* Ensure to discard campaign names with invalid type (#23229)

* Prevent concurrency when removing sites (#23230)

* Prevent warning when no email is set for current user (#23228)

* Improve region name fetching (#23235)

* Unset actionsByVisitId after consumption (#23234)

* Use https URLs (#23072)

* Use https URLs

* Build vue files

* use matomo.org instead of piwik.org in some links

* updates expected UI test file

---------

Co-authored-by: innocraft-automation <innocraft-automation@users.noreply.github.com>

* Update screenshot

* Tweak regex for host checks

---------

Co-authored-by: Marc Neudert <marc@innocraft.com>
Co-authored-by: Stefan Giehl <stefan@matomo.org>
Co-authored-by: caddoo <1169490+caddoo@users.noreply.github.com>
Co-authored-by: caddoo <matthew.caddoo@gmail.com>
Co-authored-by: innocraft-automation <innocraft-automation@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. Technical debt Issues the will help to reduce technical debt
Development

Successfully merging this pull request may close these issues.

3 participants