Skip to content
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

feature: support for Kodo #2363

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feature: support for Kodo #2363

wants to merge 2 commits into from

Conversation

printlin
Copy link
Contributor

As I mentioned in [#1923 ], the support library for Qiniu Kodo has not been merged properly. I noticed that the project structure has changed and the code related to S3 has been moved to the Tools project. I have completed these changes and also submitted PR in the tools project.
Fixes #
#1923

@kubbot
Copy link
Member

kubbot commented Jun 26, 2024

CLA Assistant Lite bot 🤖 All Contributors have signed the open-im-server CLA.
The signed information is recorded 🤖here

@printlin
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

openimbot added a commit to openim-sigs/cla that referenced this pull request Jun 26, 2024
@printlin
Copy link
Contributor Author

I noticed that the automatic check did not pass because the code for Kodo is not yet included in the tools project. I need to merge the tools project before it can be compiled properly, and I cannot follow up on this error.

@skiffer-git
Copy link
Member

Compilation aborted. failed to compile openim-cmdutils for linux_amd64: running "go build -o /home/runner/work/open-im-server/open-im-server/_output/bin/platforms/linux/amd64/openim-cmdutils /home/runner/work/open-im-server/open-im-server/cmd/openim-cmdutils/main.go" failed with exit code 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants