Devon Carew
Devon Carew
From a discussion with @leafpetersen, he saw challenges with this analysis mode (as it's implemented today?).
> I guess it is possible to disable a recommended lint in a package and certainly using From a discussion, I believe another way that was proposed to disable this...
I believe that this is working as intended. That section of the license is telling people to add copyright statements at the start of new source files. You can see...
Those repos are using a different (BSD) license. @srawlins - we'll probably want to update the contributing doc here to indicate the header we'd like for new files. Perhaps something...
From a brief look, this has to do with the span splitting one of the `\r\n` line endings.
For now I'm working around this by maintaining an 'expected fail' status file (https://github.com/devoncarew/wasmd/blob/main/tool/tests.properties), wrapping `dart test --reporter json` in a script which parses the json test reporter output (https://github.com/devoncarew/wasmd/blob/main/tool/tests.dart),...
> I just like the configuration to live with the actual test similar to how skip works. That makes sense. In my case, the tests are generated, so I would...
This sounds reasonable to me; I doubt it's critical that the toString() method identify the exact subclass of the sourcespan.
For posterity, the lint we want to enable to catch violations here is `no_runtimeType_toString` (that lint does have an issue where it doesn't report all violations: https://github.com/dart-lang/linter/issues/3635).
Yup! I'm planning on doing that today.