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 GitHub Action to build Java deploy jars #987

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jin
Copy link
Member

@jin jin commented Oct 25, 2023

Currently, we build the deploy jars under //private/tools/java/... by hand. This PR adds a new GitHub action to build and upload those JARs if the PR contains a change that's transitively depended upon by any of the JARs.

The implementation still requires us to download the zip and upload it in a new PR, but at least it's built from CI, not a local dev machine. Small improvement, that.

A possible next step is to change the upload artifact job into a create PR job, so whenever code changes for the java tools, we get an automated PR that updates the deploy jars in the repo.

Example run: https://github.com/jin/rules_jvm_external/actions/runs/6642435323

@jin jin marked this pull request as ready for review October 25, 2023 12:19
@jin jin force-pushed the prebuilts-deploy branch 23 times, most recently from 19bd785 to 610f41e Compare October 25, 2023 13:42
@jin jin force-pushed the prebuilts-deploy branch 5 times, most recently from 4fd6492 to 6881aad Compare October 25, 2023 15:10
@jin jin changed the title GitHub Actions test Add GitHub Action to build Java deploy jars Oct 25, 2023
@jin jin requested review from shs96c and cheister October 25, 2023 15:14
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.

None yet

2 participants