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

Fix/gradle run, SpringBootNewProject version pick #7

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MrNutt
Copy link

@MrNutt MrNutt commented Oct 11, 2024

Hello!
Thank you for this project and for the awesome tutorial on YouTube about setup NVim for Java development! Was looking for something similar for quite a long time.

This PR contains some tiny changes:

Fix for Gradle

Lua empty string is considered as True, so Gradle branch in the get_run_command was never reachable.
Also I'd like to suggest to use wrappers for both Maven and Gradle - I assume most SpringBoot project will have them.

Fix for SpringBootNewProject.get_boot_version

Currently version of the SpringBoot is hardcoded (3.3.1.RELEASE) and is not available on the site. I suggest to just pick one from the available values to avoid error in case when click Enter for all default values.

README update

To use :SpringBootNewProject CLI tool needs to be installed, and it was not mentioned in the README. I thought that it would be nice to mention that for new users.

I'll be glad if some of those changes might be useful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant