Skip to content

Fix various diagnostics typos in settings.mdx and diagnostics.mdx#115

Merged
carsakiller merged 3 commits intoLuaLS:mainfrom
Fringale:wiki-diagnostics-typos
Mar 17, 2026
Merged

Fix various diagnostics typos in settings.mdx and diagnostics.mdx#115
carsakiller merged 3 commits intoLuaLS:mainfrom
Fringale:wiki-diagnostics-typos

Conversation

@Fringale
Copy link
Copy Markdown
Contributor

This PR fixes the following issues in the wiki, all related to diagnostics:

  • The conventions group was missing from the default values listed for the diagnostics.groupFileStatus and diagnostics.groupSeverity settings
  • The global-element diagnostic was incorrectly referenced as global-elements in the default values listed for the diagnostics.neededFileStatus and diagnostics.severity settings
  • The cast-type-mismatch diagnostic was referenced twice in the diagnostics page: once in the type-check group (correct) and once in the luadoc group (incorrect)

The conventions group was missing from the default values listed for diagnostics.groupFileStatus and diagnostics.groupSeverity settings.
Renamed global-elements to global-element in the default values listed for diagnostics.neededFileStatus and diagnostics.severity settings.
A duplicate cast-type-mismatch diagnostic entry was incorrectly listed in the luadoc group, in addition to the type-check group.
@carsakiller carsakiller added the wiki/tweak A small fix, like grammar or typo. label Mar 17, 2026
Copy link
Copy Markdown
Collaborator

@carsakiller carsakiller left a comment

Choose a reason for hiding this comment

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

Thanks a bunch!

@carsakiller carsakiller merged commit 59518d1 into LuaLS:main Mar 17, 2026
@Fringale
Copy link
Copy Markdown
Contributor Author

Thanks for the quick review and merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wiki/tweak A small fix, like grammar or typo.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants