-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Update scripting guide #32028
Update scripting guide #32028
Conversation
🙈 The PR is closed and the preview is expired. |
@@ -149,9 +147,11 @@ For now the application does nothing new. | |||
.How do I edit this file and get content assist? | |||
==== | |||
As there is nothing but a `.java` file, most IDE's don't handle content assist well. | |||
To work around that you can run `jbang edit quarkusapp.java`, this will print out a directory that will have a temporary project setup you can use in your IDE. | |||
To work around that you can run `jbang edit quarkusapp.java`. | |||
This will create a directory, containing a temporary project setup, and will open it in https://www.jbang.dev/documentation/guide/latest/editing.html[the default editor]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in latest jbang we actually default to assume you have jbang plugin installed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated the text to only say use "jbang edit quarkusapp.java` or refer to the jbang docs for more information.
2020-08-30 06:23:36,846 INFO [io.quarkus] (main) Quarkus {quarkus-version} native started in 0.015s. Listening on: http://0.0.0.0:8080 | ||
2020-08-30 06:23:36,846 INFO [io.quarkus] (main) Profile prod activated. | ||
2020-08-30 06:23:36,846 INFO [io.quarkus] (main) Installed features: [cdi, resteasy-reactive] | ||
2023-03-22 09:38:45,450 INFO [io.quarkus] (main) quarkus 999-SNAPSHOT native (powered by {quarkus-version}) started in 0.009s. Listening on: http://0.0.0.0:8080 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2023-03-22 09:38:45,450 INFO [io.quarkus] (main) quarkus 999-SNAPSHOT native (powered by {quarkus-version}) started in 0.009s. Listening on: http://0.0.0.0:8080 | |
2023-03-22 09:38:45,450 INFO [io.quarkus] (main) quarkus {quarkus-version} native (powered by {quarkus-version}) started in 0.009s. Listening on: http://0.0.0.0:8080 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the application version, not the quarkus version.
i.e., running the example above with 3.2.0.Final results in the following output:
2023-07-10 14:49:08,347 INFO [io.quarkus] (main) quarkus 999-SNAPSHOT on JVM (powered by Quarkus 3.2.0.Final) started in 0.289s. Listening on: http://0.0.0.0:8080
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry - completely missed this. fixes are good but a few adjustment nee
8032ac0
to
a0b59dc
Compare
@maxandersen could you please have another look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All the comments were addressed.
No description provided.