bergercookie / asm-lsp

Language server for NASM/GAS/GO Assembly
https://crates.io/crates/asm-lsp
BSD 2-Clause "Simplified" License
269 stars 18 forks source link

feat: make config items optional #132

Closed WillLillis closed 2 months ago

WillLillis commented 2 months ago

This makes every item within .asm-lsp.toml optional. Leaving an item out of the config file is now the same as marking it as false.

It was fine requiring every option in the config when there were only a few, but now that we've added a few more instruction sets and assemblers, I imagine this will become/already is quite annoying to users. In addition, we can now add additional instruction sets and assemblers without making a breaking change, as missing items will no longer lead to a de-serialization failure of the TargetConfig structure from .asm-lsp.toml.