-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Remove SnakeYaml dependency #31512
Remove SnakeYaml dependency #31512
Conversation
R: @chamikaramj |
Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control |
This is motivated by breaking change of jackson-databind 2.15.x. Can we revert #31473 at the moment also. |
I don't think we should revert. #31405 was already fixed. |
Should this be closed, then? And not blocking 2.57.0? |
This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@beam.apache.org list. Thank you for your contributions. |
This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
SnakeYaml dependency was introduced in #30808 but it led to conflicts with the existing jackson YAML dependency (see #31405 and #26743 (comment))