-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
CLI: add project name and version #32567
Comments
/cc @ebullient (cli), @maxandersen (cli) |
@aloubyansky said it could be done. |
#29379 -- like this? |
No, this is about a project in which the CLI is executing a command. The challenge is to do that efficiently for Gradle projects. If we add it to |
How about we generate a json that gets updated by quarkus dev to store things that are semi static like this so you only pay the cost lazily ? |
I am not 100% sure on why is it expensive for gralde projects to get the name and version. So, I am talking out loud in hope that someone will correct me. Let's start with name:
And for version:
It doesn't seem like a huge amount of work to get the |
As long as there is no expectation for us to provide accurate info but the best effort then yes, there are options. |
@aloubyansky: Initially bringing the gradle api to devtools common didn't seem like a good idea to me, but we do bring the maven api, already. wdyt? Shall we bring in the gradle api or go with best effort? |
One difference is that the Maven resolver API is used to communicate with the registry in both Maven and Gradle projects. The other one is that with Maven API we can analyze projects "from outside", i.e. w/o requiring to launch a Maven process. The Gradle tooling API is different in that regard, which is why normally we don't bother using it preferring to invoke a Gradle task. |
Description
In order to do my CLI plugin which creates packages for Debian or RPM, I need to be able to access the project's name and version.
The current
QuarkusProject
API doesn't contain this info. Could we add it?Implementation ideas
No response
The text was updated successfully, but these errors were encountered: