Enable pkg:pedantic lints (dart-lang/markdown#236)

pkgs/markdown/.travis.yml

diff --git a/pkgs/markdown/.travis.yml b/pkgs/markdown/.travis.yml
index 88763de..7a99966 100644
--- a/pkgs/markdown/.travis.yml
+++ b/pkgs/markdown/.travis.yml
@@ -8,7 +8,7 @@
   - test: --reporter expanded
     xvfb: false
   - dartfmt
-  - dartanalyzer
+  - dartanalyzer: --fatal-warnings --fatal-infos .
 
 # Only building master means that we don't run two builds for each pull request.
 branches:

pkgs/markdown/analysis_options.yaml

diff --git a/pkgs/markdown/analysis_options.yaml b/pkgs/markdown/analysis_options.yaml
index 58a1e86..e2b8e87 100644
--- a/pkgs/markdown/analysis_options.yaml
+++ b/pkgs/markdown/analysis_options.yaml
@@ -1,3 +1,4 @@
+include: package:pedantic/analysis_options.yaml
 analyzer:
   strong-mode:
     implicit-casts: false
@@ -16,7 +17,8 @@
     - avoid_return_types_on_setters
     - await_only_futures
     - camel_case_types
-    - comment_references
+    # https://github.com/dart-lang/linter/issues/574
+    #- comment_references
     - control_flow_in_finally
     - directives_ordering
     - empty_catches

pkgs/markdown/lib/src/block_parser.dart

diff --git a/pkgs/markdown/lib/src/block_parser.dart b/pkgs/markdown/lib/src/block_parser.dart
index 0a0b678..a121927 100644
--- a/pkgs/markdown/lib/src/block_parser.dart
+++ b/pkgs/markdown/lib/src/block_parser.dart
@@ -589,7 +589,7 @@
     var childLines = <String>[];
 
     void endItem() {
-      if (childLines.length > 0) {
+      if (childLines.isNotEmpty) {
         items.add(new ListItem(childLines));
         childLines = <String>[];
       }

pkgs/markdown/lib/src/html_renderer.dart

diff --git a/pkgs/markdown/lib/src/html_renderer.dart b/pkgs/markdown/lib/src/html_renderer.dart
index 2429ced..8c65a43 100644
--- a/pkgs/markdown/lib/src/html_renderer.dart
+++ b/pkgs/markdown/lib/src/html_renderer.dart
@@ -17,7 +17,7 @@
     ExtensionSet extensionSet,
     Resolver linkResolver,
     Resolver imageLinkResolver,
-    bool inlineOnly: false}) {
+    bool inlineOnly = false}) {
   var document = new Document(
       blockSyntaxes: blockSyntaxes,
       inlineSyntaxes: inlineSyntaxes,

pkgs/markdown/lib/src/inline_parser.dart

diff --git a/pkgs/markdown/lib/src/inline_parser.dart b/pkgs/markdown/lib/src/inline_parser.dart
index 8220694..a52f42b 100644
--- a/pkgs/markdown/lib/src/inline_parser.dart
+++ b/pkgs/markdown/lib/src/inline_parser.dart
@@ -129,7 +129,7 @@
     var nodes = _stack.last.children;
 
     // If the previous node is text too, just append.
-    if (nodes.length > 0 && nodes.last is Text) {
+    if (nodes.isNotEmpty && nodes.last is Text) {
       var textNode = nodes.last as Text;
       nodes[nodes.length - 1] = new Text('${textNode.text}$text');
     } else {
@@ -499,7 +499,7 @@
   /// [emphasis delimiters]: http://spec.commonmark.org/0.28/#can-open-emphasis
   final bool requiresDelimiterRun;
 
-  TagSyntax(String pattern, {String end, this.requiresDelimiterRun: false})
+  TagSyntax(String pattern, {String end, this.requiresDelimiterRun = false})
       : endPattern = new RegExp((end != null) ? end : pattern, multiLine: true),
         super(pattern);
 
@@ -587,7 +587,7 @@
 
   final Resolver linkResolver;
 
-  LinkSyntax({Resolver linkResolver, String pattern: r'\['})
+  LinkSyntax({Resolver linkResolver, String pattern = r'\['})
       : this.linkResolver = (linkResolver ?? (String _, [String __]) => null),
         super(pattern, end: r'\]');
 
@@ -1184,7 +1184,7 @@
     parser._stack.removeLast();
 
     // If the stack is empty now, this is the special "results" node.
-    if (parser._stack.length == 0) return children;
+    if (parser._stack.isEmpty) return children;
     var endMatchIndex = parser.pos;
 
     // We are still parsing, so add this to its parent's children.

pkgs/markdown/pubspec.yaml

diff --git a/pkgs/markdown/pubspec.yaml b/pkgs/markdown/pubspec.yaml
index feb9d45..0944a5c 100644
--- a/pkgs/markdown/pubspec.yaml
+++ b/pkgs/markdown/pubspec.yaml
@@ -9,7 +9,7 @@
   markdown:
 
 environment:
-  sdk: '>=2.0.0-dev <3.0.0'
+  sdk: '>=2.0.0 <3.0.0'
 
 dependencies:
   args: ^1.0.0
@@ -24,5 +24,6 @@
   html: '>=0.12.2 <0.14.0'
   js: ^0.6.1
   path: ^1.3.1
+  pedantic: ^1.3.0
   test: ^1.2.0
   yaml: ^2.1.8

pkgs/markdown/test/util.dart

diff --git a/pkgs/markdown/test/util.dart b/pkgs/markdown/test/util.dart
index 2c87471..af84655 100644
--- a/pkgs/markdown/test/util.dart
+++ b/pkgs/markdown/test/util.dart
@@ -43,7 +43,7 @@
     Iterable<InlineSyntax> inlineSyntaxes,
     Resolver linkResolver,
     Resolver imageLinkResolver,
-    bool inlineOnly: false}) {
+    bool inlineOnly = false}) {
   test(description, () {
     var result = markdownToHtml(markdown,
         blockSyntaxes: blockSyntaxes,

pkgs/markdown/tool/stats_lib.dart

diff --git a/pkgs/markdown/tool/stats_lib.dart b/pkgs/markdown/tool/stats_lib.dart
index 5375666..5622bed 100644
--- a/pkgs/markdown/tool/stats_lib.dart
+++ b/pkgs/markdown/tool/stats_lib.dart
@@ -80,9 +80,9 @@
 enum CompareLevel { strict, loose, fail, error }
 
 CompareLevel compareResult(Config config, CommonMarkTestCase expected,
-    {bool throwOnError: false,
-    bool verboseFail: false,
-    bool verboseLooseMatch: false}) {
+    {bool throwOnError = false,
+    bool verboseFail = false,
+    bool verboseLooseMatch = false}) {
   String output;
   try {
     output =