forbidigo icon indicating copy to clipboard operation
forbidigo copied to clipboard

Include type info for unknown packages

Open isergieienkov opened this issue 11 months ago • 2 comments

builtin package is non detected (maybe some others too) which results in missing both package and type info in match string. I propose to keep as much info as possible. Should allow matching of builtin type functions too. For example: var err error err.Error() -----> results in .Error match string. With this change it will be .error.Error

isergieienkov avatar Mar 05 '24 16:03 isergieienkov

@isergieienkov Thanks for your proposal and sorry for not reviewing it sooner. Is it possible to write a test for this behavior so I can better understand the issue? As I understand it, you'd like to forbid some identifiers in the builtin package. Thanks.

ashanbrown avatar Apr 08 '24 04:04 ashanbrown

Added test for this case

isergieienkov avatar Apr 11 '24 11:04 isergieienkov