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

Handle panic on missing search-paths #32

Merged
merged 2 commits into from
Jul 31, 2024

Conversation

mkatychev
Copy link
Contributor

The config below will panic on an .unwrap()

  openscad_lsp = {
    settings = {
      openscad = {
        fmt_style = "file",
        fmt_exe= "clang-format",
        -- search_paths = "/libs", this will panic if left commented
      },
    },
  }

modified unwrap to be unwrap_or_default

@Leathong Leathong merged commit 419a4b2 into Leathong:master Jul 31, 2024
3 of 4 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants