Skip to content

Conversation

@ericluoliu
Copy link
Contributor

Note: this is an updated version of a previously closed PR (#153) that I cluttered with poor commits. Apologies for cluttering the list of Closed PRs.

  • Why submit this pull request?
  • Bugfix
  • New feature provided
  • Improve performance

New feature or improvement

Updated pom.xml files to allow deployment of JAR to Maven Central Repository. This will allow users to download the JAR from the Maven Repository and build their plugins without having to clone the source code.

I have deployed a test JAR to Maven Central (https://search.maven.org/search?q=io.github.ericluoliu) under my GAV coordinates, and I have used this JAR by referencing it as a dependency in pom.xml to build a functional (end-to-end) demo plugin in a new Java SpringBoot project.

-Source branch: deployJAR branch on https://github.com/ericluoliu/apisix-java-plugin-runner

-Related commits and pull requests: N/A

-Target branch: main branch on https://github.com/apache/apisix-java-plugin-runner

@tzssangglass tzssangglass self-requested a review June 28, 2022 04:09
@tzssangglass tzssangglass self-assigned this Jun 28, 2022
@tzssangglass
Copy link
Member

LGTM.

I will give this PR a validation as soon as possible and then add MAINTAIN.md and the corresponding CI (I hope the push jar can be done in the CI)

@tzssangglass tzssangglass changed the title feat: deploy jar to the Maven Central repository feat: support deploy jar to the Maven Central repository Jun 30, 2022
@tzssangglass tzssangglass merged commit e0d3548 into apache:main Jun 30, 2022
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.

2 participants