-
Notifications
You must be signed in to change notification settings - Fork 22
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
Jolokia 2.1 #1111
Jolokia 2.1 #1111
Conversation
Test resultsRun attempt: 2322
|
@mmuzikar may I ask for help? Is there a known problem in hawtio e2e tests, or did I break something by removing jquery? |
@grgrzybek |
@mmuzikar I only saw that after trying to run
to:
|
darn... ES modules vs. node |
@grgrzybek Btw, can you please follow the commit message convention? Otherwise, the commit-and-tag-version cannot pick up the changes correctly for automated releases. |
ah, sorry - old habits... |
850b3ac
to
ef09ff9
Compare
ef09ff9
to
dab6b29
Compare
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.
LGTM.
Just curious about what was the culprit of the last e2e issue?
Great job! |
it was silly mistake related to switch from We had this:
and I simply removed it. The fix was simply to actually set up the credentials to be passed directly to
|
No description provided.