fix: remove okhttp1 due to conflicts risk #294

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.

Sign up for GitHub

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

Merged

simon824 merged 2 commits into master from exclude-okhttp1

Dec 13, 2023

Member

imbajin commented Dec 12, 2023 •

edited

Loading

Purpose of the PR

same as fix: exclude okhttp1 which has different groupID with okhttp3 incubator-hugegraph-toolchain#555

Main Changes

Verifying these changes

Trivial rework / code cleanup without any test coverage. (No Need)
Already covered by existing tests, such as (please modify tests here).
Need tests and can be verified as follows.

Does this PR potentially affect the following parts?

Nope
Dependencies (add/update license info)
Modify configurations
The public API
Other affects (typed here)

Documentation Status

Doc - TODO
Doc - Done
Doc - No Need

fix: remove okhttp1 due to conflicts risk

9c42d94

imbajin requested review from coderzc and javeme

December 12, 2023 13:57

Update known-dependencies.txt

0c42d43

JackyYangPassion approved these changes

View reviewed changes

simon824 approved these changes

View reviewed changes

simon824 merged commit 8580a2e into master

simon824 deleted the exclude-okhttp1 branch

December 13, 2023 00:58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment