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

Diagnostic messages does't show any valid values when an invalid value is given for a variable in server.env #313

Open
JoshwinThomasIBM opened this issue Oct 18, 2024 · 2 comments · May be fixed by #339
Labels
enhancement New feature or request

Comments

@JoshwinThomasIBM
Copy link

When an invalid value is given for a property the following diagnostic message is shown
image
Here it does't mention any valid values

however on hover over the property it displays the valid values as below
image

it will be more helpful to show the valid values along with the diagnostic message if an invalid input is given by the user , he might not be aware about the hover message over the property.

Alternatively if quick fixes are available along with the diagnostic messages then it can also be a possible workaround

@JoshwinThomasIBM JoshwinThomasIBM added the enhancement New feature or request label Oct 18, 2024
@JoshwinThomasIBM JoshwinThomasIBM changed the title Diagnostic messages does't show any valid values when an invalid value is given for a property in server.env Diagnostic messages does't show any valid values when an invalid value is given for a variable in server.env Oct 18, 2024
@JoshwinThomasIBM JoshwinThomasIBM transferred this issue from OpenLiberty/liberty-tools-vscode Oct 21, 2024
@JoshwinThomasIBM
Copy link
Author

This behaviour is uniform across all the 3 IDE's

@cherylking
Copy link
Member

If we address this, I would recommend providing a quick fix with each valid value for those variables that have predefined values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging a pull request may close this issue.

2 participants