Skip to content

Conversation

@avinxshKD
Copy link
Contributor

Adds Maven Central publishing configuration to the PDF library.

  • Added maven-publish plugin
  • Added publishing block for local app repository
  • Added mavenPublishing block with POM metadata
  • Follows same pattern as preprocessor module

Part of #1403

@avinxshKD
Copy link
Contributor Author

@Stefterv hey started with PDF since it was the easiest.
Let me know if this looks good and I'll work on Net/SVG next. thanks

}

mavenPublishing{
publishToMavenCentral(SonatypeHost.CENTRAL_PORTAL, automaticRelease = true)
Copy link
Collaborator

Choose a reason for hiding this comment

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

please set the coordinates like in my PR

Copy link
Collaborator

Choose a reason for hiding this comment

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

version is available as a variable here so you can use the exact line as in my pr and only replace the library name

@Stefterv Stefterv changed the title publish pdf library Publish org.processing.core.pdf to maven Central Feb 4, 2026
@Stefterv
Copy link
Collaborator

Stefterv commented Feb 4, 2026

Thanks @avinxshKD I added a few comments ;)

@avinxshKD
Copy link
Contributor Author

Thanks @avinxshKD I added a few comments ;)

sure was already on it thanks and welcome :)

@avinxshKD
Copy link
Contributor Author

@Stefterv Updated the coordinates and removed the signing checkk. and
hve skipped the optional migration for this PR to keep it focused on just enabling publishing. Happy to look into a shared plugin in a follow-up refactor for all libraries at once, what do you say?

@avinxshKD
Copy link
Contributor Author

Ready for another look.

@Stefterv
Copy link
Collaborator

Stefterv commented Feb 4, 2026

The coordinates still need to be fixed, the version should be equal to the processing version, not the version from the library.properties

@avinxshKD
Copy link
Contributor Author

@Stefterv hve updated the version to use rootProject.version so it matches the Processing version as requested

@Stefterv
Copy link
Collaborator

Stefterv commented Feb 4, 2026

Hi @avinxshKD, it appears to me that you are overly relying on AI for making your changes, please take a look at this.
If not, please copy the coordinates line directly from my pr, only changing the library name

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