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

Improve CLI install instructions #23134

Merged
merged 2 commits into from
Mar 9, 2022
Merged

Conversation

maxandersen
Copy link
Member

Details in quarkusio/quarkusio.github.io#1307
on why the "trust" call is added.

I moved jbang up as it is the only one that works on every OS and without
java first installed. Thus absolute fastest/simplest to use on all major OS's.

And moved homebrew down as that is only available for mac/osx.

@quarkus-bot
Copy link

quarkus-bot bot commented Jan 24, 2022

Thanks for your pull request!

The title of your pull request does not follow our editorial rules. Could you have a look?

  • title should preferably start with an uppercase character (if it makes sense!)

This message is automatically generated by a bot.

* <<sdkman,SDKMAN!>>
* <<jbang,JBang>> for Linux, OSX and Windows (no java required)
* <<sdkman,SDKMAN!>> for Linux and OSX (java required)
* <<homebrew,Homebrew>> for OSX (java required)

Copy link
Member

@ebullient ebullient Jan 24, 2022

Choose a reason for hiding this comment

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

While I agree with moving Homebrew down, I would perhaps leave SDKMan first unless you have resolved the trusted source issue with JBang, as that is confusing users.

Have you fixed the warning for JBang (re: maven central as source or some other means)? Edit: I see the other PRs now, and I have a comment on one of them.

Copy link
Member

Choose a reason for hiding this comment

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

I decided to make them alphabetically sorted. And I would prefer we keep it that way. There's absolutely no way we can find an ordering that will fit all the users so alphabetical is probably the best.

Copy link
Contributor

Choose a reason for hiding this comment

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

Is it OSX or macOS? Also make sure Java is capitalized

Copy link
Contributor

Choose a reason for hiding this comment

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

I agree with @ebullient on leaving SDKMan first. IMHO we should promote what works better and faster first instead of alphabetically.


On Linux, macOS, and Windows (using WSL or bash compatible shell like Cygwin or MinGW)
[source,bash]
----
curl -Ls https://sh.jbang.dev | bash -s - trust add https://repo1.maven.org/maven2/io/quarkus/quarkus-cli/
Copy link
Member

@ebullient ebullient Jan 24, 2022

Choose a reason for hiding this comment

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

Can we add a comment re: what this does?

This step should be separated out and (briefly) explained. Ubuntu uses words like this to explain adding additional repos/keys (using installation of mongo as an example):

From a terminal, issue the following command to import the MongoDB public GPG Key from ...

wget -qO - https://www.mongodb.org/static/pgp/server-5.0.asc | sudo apt-key add -

The operation should respond with an OK code.

Something like this (but about adding Quarkusio as a trusted source) would suffice.

Comment on lines 20 to 22
* <<jbang,JBang>> for Linux, OSX and Windows (no java required)
* <<sdkman,SDKMAN!>> for Linux and OSX (java required)
* <<homebrew,Homebrew>> for OSX (java required)
Copy link
Contributor

@gastaldi gastaldi Feb 23, 2022

Choose a reason for hiding this comment

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

Suggested change
* <<jbang,JBang>> for Linux, OSX and Windows (no java required)
* <<sdkman,SDKMAN!>> for Linux and OSX (java required)
* <<homebrew,Homebrew>> for OSX (java required)
* <<jbang,JBang>> for Linux, macOS and Windows (no Java required)
* <<sdkman,SDKMAN!>> for Linux and macOS (Java required)
* <<homebrew,Homebrew>> for macOS (Java required)

Details in quarkusio/quarkusio.github.io#1307
on why the "trust" call is added.

I moved jbang up as it is the only one that works on every OS and without
java first installed. Thus absolute fastest/simplest to use on all major OS's.

And moved homebrew down as that is only available for mac/osx.
@gsmet gsmet self-assigned this Mar 9, 2022
@gsmet
Copy link
Member

gsmet commented Mar 9, 2022

I think I came up with an acceptable compromise and also used tabs to present the alternatives.

I will merge once CI is green and push it to the website.

@gsmet gsmet dismissed ebullient’s stale review March 9, 2022 11:25

Somehow addressed.

@gsmet gsmet changed the title fix cli install Improve CLI install instructions Mar 9, 2022
@gsmet gsmet merged commit 604dec9 into quarkusio:main Mar 9, 2022
@quarkus-bot quarkus-bot bot added this to the 2.8 - main milestone Mar 9, 2022
@gsmet gsmet modified the milestones: 2.8 - main, 2.7.4.Final Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants