Add a better error message for invalid 'resources' #305
Conversation
When 'resources' field has something other than `Str` to safely match against, e.g. a Pair, it threw with `Invocant of method 'match' must be a type object of type 'Any', not an object instance of type 'Pair'.`, which is not helpful. Here we explicitly check for each resource item to be a Str and provide an error message that is easier to understand.
|
fwiw I don’t put META6 validation inside of zef. zef is not an authoring tool and thus not intended to catch authoring issues. If I were to handle these issues it would require a full blown META validation module, and at that point might as well supersede both Test::META and Mi6. The only way I’d probably add this type of thing to zef is if Rakudo/CUR end up creating a META6 validator... because remember one can install modules without zef (and will get just as poor error messages). See: #251 (comment) |
|
I see, the issue is more complex than it seem at first, thanks. I am withdrawing the PR and will look at validation modules specified. Sorry for the time taken. |
|
Note that an This is nice because it acts as documentation for the format as well as validating it on the ecosystem side. |
It caused some confusion at https://colabti.org/irclogger/irclogger_log/perl6?date=2019-06-08#l139