fix: clarify app setup messages and hint about jbang version --update - #2558
Conversation
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Not sure whether you like my proposal at maxandersen#9 |
When JBang is already on PATH, clarify that --force updates shell environment files (not JBang itself) and suggest running 'jbang version --update' to update JBang. Fixes jbangdev#2067 Co-authored-by: Oliver Kopp <kopp.dev@gmail.com>
|
sorry @koppor didnt see it was you opened PR on my fork not jbangdev/jbang :) added you as co-author on this one. thanks! |
|
Thank you. In that moment I totally forgot that there was the suggestion functionality 😅 |
When
jbang app setupdetects JBang is already on PATH, the messages now:--forceupdates shell environment files (not JBang itself)jbang version --updateto update JBang itselfBefore:
After:
Fixes #2067