Skip to content

Publish PR API and dev bundles#12672

Merged
jpenilla merged 13 commits intomainfrom
feat/pr-publishing
Jun 17, 2025
Merged

Publish PR API and dev bundles#12672
jpenilla merged 13 commits intomainfrom
feat/pr-publishing

Conversation

@jpenilla
Copy link
Member

@jpenilla jpenilla commented Jun 15, 2025

You'll need to setup a fork and a test PR like jpenilla#7 to test changes, due to use of workflow completion triggers (the action under test must be on the main branch)

We will need to rename the build-pr-jar tag to publish-pr (it controls Maven publishing and Paperclip posting)

Action handling upload and push to GH packages:
https://github.com/PaperMC/action-pr-publishing

CF worker to proxy unauthenticated requests
https://github.com/PaperMC/packages-maven-mirror

Current test app permissions (not sure if all are required, just use papermc.io or smth for the homepage and other required links):
Screenshot 2025-06-14 at 10 56 18 PM

Screenshot 2025-06-15 at 6 47 33 PM

@github-project-automation github-project-automation bot moved this to Awaiting review in Paper PR Queue Jun 15, 2025
@jpenilla jpenilla changed the base branch from main to dev/1.21.6 June 15, 2025 04:36
@jpenilla jpenilla force-pushed the feat/pr-publishing branch from 84b1454 to bfac8c1 Compare June 16, 2025 20:10
@jpenilla jpenilla changed the base branch from dev/1.21.6 to main June 16, 2025 20:10
@jpenilla jpenilla marked this pull request as ready for review June 17, 2025 00:23
@jpenilla jpenilla requested a review from a team as a code owner June 17, 2025 00:23
@github-project-automation github-project-automation bot moved this from Awaiting review to Awaiting final testing in Paper PR Queue Jun 17, 2025
@jpenilla jpenilla merged commit 39203a6 into main Jun 17, 2025
5 checks passed
@jpenilla jpenilla deleted the feat/pr-publishing branch June 17, 2025 00:35
@github-project-automation github-project-automation bot moved this from Awaiting final testing to Merged in Paper PR Queue Jun 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

2 participants