apache / incubator-hugegraph-commons

(Archived Warning)Please visit "apache/hugegraph" repo instead
https://github.com/apache/hugegraph/tree/master/hugegraph-commons
Apache License 2.0
29 stars 45 forks source link

feat(common): replace jersey dependencies with OkHttp (Breaking Change) #133

Closed zhenyuT closed 9 months ago

zhenyuT commented 1 year ago

之前在tool-chains提过一个pr,直接修改client的代码将jersey使用okhttp替换

现在尝试直接修改common的代码,不过common的代码中单元测试和jersey是强相关的,在这一块遇到一些问题,然后尝试自己修改单测代码进行跑通。不过由于对单测的代码不太熟悉,对于代码的意图不太了解,麻烦社区同学帮忙看看

Breaking Changes


Other repo/users could refer the info to update/adapt the new version, and comment in this PR if anything missed

codecov[bot] commented 1 year ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (b49be05) 93.15% compared to head (81aa7ca) 93.15%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #133 +/- ## ========================================= Coverage 93.15% 93.15% Complexity 65 65 ========================================= Files 9 9 Lines 263 263 Branches 22 22 ========================================= Hits 245 245 Misses 8 8 Partials 10 10 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 11 months ago

Due to the lack of activity, the current pr is marked as stale and will be closed after 180 days, any update will remove the stale label