Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix NPE in jline3 Example as ConfigurationPath cannot be null anymore #2148

Merged
merged 2 commits into from
Nov 12, 2023

Conversation

llzen44
Copy link
Contributor

@llzen44 llzen44 commented Nov 11, 2023

ConfigurationPath cannot be null anymore since commit 9706ead in jline3. An Objects.requireNonNull check now requires a valid ConfigurationPath.

Copy link
Owner

@remkop remkop left a comment

Choose a reason for hiding this comment

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

That looks good.
Can you also update the picocli-shell-jline3/README.md file, which shows the same code?
Thank you!

@remkop remkop added this to the 4.7.6 milestone Nov 12, 2023
@llzen44
Copy link
Contributor Author

llzen44 commented Nov 12, 2023

Sure thing. Done
Thanks for the review

@llzen44 llzen44 requested a review from remkop November 12, 2023 10:07
@remkop remkop added type: bug 🐛 theme: shell An issue or change related to interactive (JLine) applications type: doc 📘 labels Nov 12, 2023
@remkop remkop merged commit a15588b into remkop:main Nov 12, 2023
0 of 40 checks passed
@remkop
Copy link
Owner

remkop commented Nov 12, 2023

Merged. Thank you for the contribution!

remkop added a commit that referenced this pull request Nov 12, 2023
vorburger added a commit to vorburger/picocli that referenced this pull request Dec 15, 2023
vorburger added a commit to vorburger/picocli that referenced this pull request Dec 15, 2023
remkop pushed a commit that referenced this pull request Dec 16, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
theme: shell An issue or change related to interactive (JLine) applications type: bug 🐛 type: doc 📘
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants