Skip to content

ci: update setup-node and pin Ubuntu runner - #939

Merged
jsuchal merged 2 commits into
mainfrom
chore/stabilize-ci-runtime
Oct 23, 2026
Merged

jsuchal merged 2 commits into
mainfrom
chore/stabilize-ci-runtime

Conversation

@jsuchal

@jsuchal jsuchal commented Oct 23, 2026

Copy link
Copy Markdown
Member

The problem

CI upozorňuje na deprecated Node.js 20 runtime v actions/setup-node@v4 a pripravovaný prechod ubuntu-latest na Ubuntu 26. Nejde o zlyhanie testov, ale automatický prechod runnera by zmenil prostredie bez samostatného overenia.

Solution

Aktualizovať actions/setup-node na v6 (Node.js 24 action runtime) a všetky štyri joby pripnúť na ubuntu-24.04. Aplikačný Node zostáva na verzii 22. Prechod runnera na Ubuntu 26 je mimo rozsahu.

Overenie: GitHub Actions — Run tests, Brakeman a FOP Saxon check. Push to GitLab zostáva obmedzený na main.

Demo

Bez vizuálnej zmeny; ide iba o konfiguráciu CI. Výsledky kontrol budú dostupné priamo v PR.

@jsuchal
jsuchal requested a review from a team as a code owner October 23, 2026 07:19
@celuchmarek

Copy link
Copy Markdown
Member

@jsuchal ten node rozumiem. Ale ten runner image, idk. Ono v Dockerfile produkčnom je aj tak Debian 13, čiže ten pinning runnera tu nič neprináša. Chceš to aj tak pinnúť?

@jsuchal

jsuchal commented Oct 23, 2026

Copy link
Copy Markdown
Member Author

@jsuchal ten node rozumiem. Ale ten runner image, idk. Ono v Dockerfile produkčnom je aj tak Debian 13, čiže ten pinning runnera tu nič neprináša. Chceš to aj tak pinnúť?

pravda toto som si ani nevsimol, je to prec

@jsuchal
jsuchal merged commit e49b176 into main Oct 23, 2026
7 of 8 checks passed
@jsuchal
jsuchal deleted the chore/stabilize-ci-runtime branch October 23, 2026 11:18
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.

2 participants