-
Notifications
You must be signed in to change notification settings - Fork 124
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
minor dependency updates + test-summary workflow action #119
Conversation
5079c55
to
ce1ceb1
Compare
f22f88b
to
57e61ef
Compare
ok thats weird. I can't explain right now why this PR started failing. Can't reproduce this issue locally and I basically reverted everything already. |
a36cf44
to
e40f041
Compare
there you go. Some test annotations were missing. Whats interesting however is that the tests run in a different order on github than they do on my system locally. Once the MediaFile test started running first, it failed since it didn't boostrap roller. |
some notes on dependency bumps:
|
- use the geckodriver of the test runner - enable xserver for headful integration testing - strip repo credentials after checkout just to be sure
- asm - commons text - eclipse link - apache lucene - spring - jquery webjars - various maven plugins
e40f041
to
f60e75f
Compare
tested locally |
<rome.version>1.18.0</rome.version> | ||
<rome.version>1.19.0</rome.version> <!-- locked in place since next version removes popono --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
another lib which is now locked since rome popono is gone
all green. @snoopdave approved via dev list -> merging |
It writes test reports directly to the workflow summary page (not to checks, which felt like a workaround anyway).
1920 in the test matrixtested locally.. lets see if everything builds with CI too