Devonte

Results 86 comments of Devonte

I'm running `crystal tool unreachable spec/spec_helper.cr -i spec/*` which lists everything as unreachable. I've also tried specifying all spec files with `-i` but it seems it doesn't accept multiple arguments.

@straight-shoota can you elaborate on what specifically? All the methods depend on the regex which is already tested in specs.

@beta-ziliani Thank you for your acknowledgement. I don't mind having several rounds of reviews, but I do find these recent ones a bit unnecessary. I understand and support having full...

`ameba -u` working great! I have a list of rules and the versions they were added in if you need it, it goes back as far as 1.3.0. All that...

Is there anything left to be added for this to be merged?

Focusing on the language side of things, I'd agree that multi-line quote strings probably shouldn't be a thing. I can't say I've ever had as use case for it that...

I'm not sure if this is the same issue but running `/bar/` in the interpreter produces this: ```sh undefined reference to `pcre2_compile_8' (Crystal::Loader::LoadError) from D:\a\crystal\crystal\src\compiler\crystal\loader.cr:76 in 'find_symbol' from D:\a\crystal\crystal\src\compiler\crystal\interpreter\compiler.cr:1885 in...

I've encountered this in 2 of my projects recently: - https://github.com/devnote-dev/cling/actions/runs/11753866125/job/32747205715#step:4:88 - https://github.com/devnote-dev/docr/actions/runs/11759771854/job/32759616408#step:3:90

I'm not sure I understand the changes here: you can already access the invalid/missing option via the `invalid_option` and `missing_option` handler methods respectively, what need is there to have this...

This is more a limitation of `OptionParser`, as far as I'm aware it doesn't parse options contextually so your only options (pun intended) are to do additional input checks in...