-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Fix jackson integration test #5997
Conversation
Weird that CI hasn't even picked this up... |
CI wasn't triggered for now so let's wait if Azure gets triggered or I'll force push this week-end. |
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
Sorry, I didn't pay enough attention to the number of passed checks. This shouldn't have been merged. I'll keep an eye on CI to make sure everything's OK. |
CI is broken on master, I submitted #6000 to fix it. I'll never ever merge something that late and without checking twice the CI checks list! Again, sorry about all this :( |
No worries! It was a bit of a special case with CI completely failing to start... Thanks for fixing in the follow up PR |
No description provided.