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

Add Java Support #52

Merged
merged 6 commits into from
Apr 20, 2020
Merged

Add Java Support #52

merged 6 commits into from
Apr 20, 2020

Conversation

diaozxin007
Copy link
Contributor

add java support

@CLAassistant
Copy link

CLAassistant commented Apr 18, 2020

CLA assistant check
All committers have signed the CLA.

@huan
Copy link
Member

huan commented Apr 19, 2020

Great to see your PR with the Java support, it will be a really huge step for us that we can support Java developers!

Could you please create an issue for some basic Java GRPC related information, and what we should get to know and next to do with our Java GRPC support with this PR?

I'd like to learn from you more from that issue, and after I understand it, I will be able to merge this PR and move forward for the final settings for supporting the Java ecosystem by creating the java package in our Github org wechaty.

Thank you very much, and I hope you have a great weekend!

P.S. Also appreciate for your efforts of the Maven Central Repo creating.

Link to https://issues.sonatype.org/browse/OSSRH-56843
Link to #53

pom.xml Outdated Show resolved Hide resolved
pom.xml Outdated
<groupId>io.github.wechaty</groupId>
<artifactId>wechaty-grpc-proto</artifactId>
<packaging>jar</packaging>
<version>1.0.0</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this version number is the module number of our Java GRPC.

It should be sync to the exact version in our package.json.

I'm not sure the workflow between the GitHub and the Marven Central Repo, it will be great if this can be archived via the DevOps workflow (GitHub Actions in our case)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix too

pom.xml Outdated Show resolved Hide resolved
proto/wechaty/puppet.proto Outdated Show resolved Hide resolved
Copy link
Member

@huan huan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much for adding the Java Maven publishing support to our GRPC module, it's so great to see that we are pushing the Java language moving forward!

I have some questions and suggestions to this PR, let's discuss about it so that we can be able to merge it after everything has been clear.

@huan huan mentioned this pull request Apr 19, 2020
@huan
Copy link
Member

huan commented Apr 20, 2020

LGTM. And it also passed our GitHub Actions for Java Building!

Cheers, and thank you very much for your effort on our Java GRPC module!

@huan huan merged commit 283c887 into wechaty:master Apr 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants