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

feat: Publish Maven BOM to improve dependency management #11

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

Conversation

reneleonhardt
Copy link
Contributor

No description provided.

@reneleonhardt
Copy link
Contributor Author

A BOM makes importing much easier, then a GitHub Action could automatically publish all artifacts when a new commit has been tagged.
Now every version of every artifact has to be specified in every project, even if you just need the "latest" (corresponding to the tree-sitter root artifact).
As the artifacts get no tags of their own, keeping up with Maven publications is no easy feat 😅
https://github.com/carlrobertoh/CodeGPT/blob/master/codegpt-treesitter/build.gradle.kts

I had to remove some old modules from settings.gradle, because the publish plugin would just include them in the BOM anyway.

@reneleonhardt
Copy link
Contributor Author

Example build: pom-default.xml.zip

@bonede
Copy link
Owner

bonede commented Apr 15, 2024

Hi, thanks for your contribution, but I prefer to keep the build script simple for now.

@reneleonhardt
Copy link
Contributor Author

Okay. I added 7 code lines in the build script and removed 4 from settings... not simple enough 😅

@sepatel
Copy link

sepatel commented Aug 4, 2024

I'd very much appreciate a bom. Any status on being able to get this moving forward?

@reneleonhardt
Copy link
Contributor Author

@sepatel Thank you for your interest, the functionality is finished, but it's not my project, I can't merge it 😅

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