surahman commented on pull request #3754: URL: https://github.com/apache/incubator-heron/pull/3754#issuecomment-1005137197
> Ok, rebased off of the new `master` branch. Updated the pinned dependencies. I think this is ready for testing. 👍🏼 > FYI, the unit tests you added already helped catch a build issue with this update. We had `org.yaml:snakeyaml` explicitly defined. The older version didn't work. I updated to the latest `1.30`. But I also tried removing the explicit dependency and it resolved to `1.29`. I'm leaning towards removing the entry and letting the Maven dependency info drive which version is selected. This might be better for maintaining going forward. If you agree, I'll remove the mention of `snakeyml` in the dependencies list. Glad they are catching issues before they make their way into production. > I believe there are other dependencies listed that no longer need to be listed. They were in the list before we had actual transitive dependency resolution logic helping us. > > _Edit: I talked myself into it. I removed the direct mention of snakeyml dependency._ Good move, hardcoded dependencies can get forgotten over time. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
