Skip to content

SPOC-939203: Bump package versions#738

Merged
lkreimann merged 4 commits into
mainfrom
feature/SPOC-939203
Mar 9, 2026
Merged

SPOC-939203: Bump package versions#738
lkreimann merged 4 commits into
mainfrom
feature/SPOC-939203

Conversation

@lkreimann

@lkreimann lkreimann commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hier hab ich nur die Einrückung angepasst, weil meine IDE das rot markiert hatte

@lkreimann lkreimann marked this pull request as ready for review March 4, 2026 06:42
Comment thread package.json Outdated
"eslint-plugin-github": "6.0.0",
"eslint-plugin-jest": "29.15.0",
"jest": "30.2.0",
"jest-circus": "29.7.0",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hier müsste auch noch auf v30 geupdated werden: https://www.npmjs.com/package/jest-circus?activeTab=versions

@dloetzke

dloetzke commented Mar 4, 2026

Copy link
Copy Markdown
Member

Ich bin etwas verwirrt, ob du oben den falschen Link angegeben hast, bei "Testen können wir das nach dem Merge mit"?

Funktioniert die bisherige Jest-Config tatsächlich? Ich dachte, man muss nun auf eine "Flat Config" umsteigen ab v30. 🤔

@lkreimann

Copy link
Copy Markdown
Contributor Author

Ich bin etwas verwirrt, ob du oben den falschen Link angegeben hast, bei "Testen können wir das nach dem Merge mit"?

Funktioniert die bisherige Jest-Config tatsächlich? Ich dachte, man muss nun auf eine "Flat Config" umsteigen ab v30. 🤔

Oh das mit Jest war mir nicht bewusst - Tests werden hier im Check ja mit ausgeführt und da wurde nicht gemeckert, daher hätte ich jetzt keine Probleme vermutet.

Der Link ist richtig - in dem Workflow nutzen wir diese Action hier (etwas unnötig leider, weil es ein Ubuntu-Runner ist aber naja :D )

@dloetzke

dloetzke commented Mar 4, 2026

Copy link
Copy Markdown
Member

Achsooo, jetzt verstehe ich das mit dem Link 😄
Ja gut, wenns läuft dann läufts. 👍🏻

Dann fehlt denke ich nur noch jest-circus (siehe oben).

@lkreimann lkreimann marked this pull request as draft March 4, 2026 10:14
@lkreimann lkreimann marked this pull request as ready for review March 6, 2026 06:48
@lkreimann

Copy link
Copy Markdown
Contributor Author

Achsooo, jetzt verstehe ich das mit dem Link 😄 Ja gut, wenns läuft dann läufts. 👍🏻

Dann fehlt denke ich nur noch jest-circus (siehe oben).

Ist jetzt auch erledigt :)

@lkreimann lkreimann merged commit 8889551 into main Mar 9, 2026
6 checks passed
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