Mark generated code as ignored by linter

[patkujawa-wf]

Unless we can get all the generated code to abide by all the rules.

See dart-lang/sdk#57261 for some options. Perhaps just // ignore_for_file: linter_rule in each generated file?

[davidmarne]

So in the dart2 dev branch you can set this by setting the header option in your build.yaml

targets:
  $default:
    builders:
      built_value_generator|built_value:
        enabled: false
    builders:
      built_redux|built_redux:
        options:
          header: // ignore_for_file: linter_rule

However, that will only work with dart 2.

If there are any specific lints you are having issues with post them here and I can just add them to all generated files for stable line.

[patkujawa-wf]

From https://github.com/Workiva/text_doc_client/pull/2242:

annotate_overrides
avoid_as
overridden_fields

[davidmarne]

will add tonight

[zoechi]

@patkujawa-wf isn't aviod_as outdated, especially with Dart 2?

[patkujawa-wf]

Not everyone is on Dart 2 yet ;)

[davidmarne]

Sorry was busy last night, i shouldn't be making promises :)

Will try add this weekend.

[patkujawa-wf]

No rush, but thanks Dave!

[kostaa]

I have tried setting the header option in the build.yaml but getting an exception from the parser.
I am running it with Dart 2.0.0-dev.54.0 version.

D:\repo__at\_moneyvore_rdx>flutter packages pub run build_runner build --delete-conflicting-outputs
[INFO] Generating build script...
Unhandled exception:
Error on line 9, column 38: Mapping values are not allowed here. Did you miss a colon earlier?
          header: /// ignore_for_file: linter_rule
                                     ^
#0      Scanner._fetchValue (package:yaml/src/scanner.dart:713:9)
dart-lang/sdk#57147      Scanner._fetchNextToken (package:yaml/src/scanner.dart:457:11)
dart-lang/sdk#57148      Scanner._fetchMoreTokens (package:yaml/src/scanner.dart:339:7)
dart-lang/sdk#57149      Scanner.peek (package:yaml/src/scanner.dart:317:27)
dart-lang/sdk#57150      Parser._parseBlockMappingKey (package:yaml/src/parser.dart:418:26)

[zoechi]

@kostaa Have you tried wrapping the string with quotes?

header: "/// ignore_for_file: linter_rule"

[kostaa]

Thanks using quotes helps but you also have to drop the second builders: key.

targets:
  $default:
    builders:
      built_value_generator|built_value:
        enabled: false
      built_redux|built_redux:
        options:
          header: "// ignore_for_file: linter_rule"

[davidmarne]

#82