Bail early in API analysis if no project is set.

See: dart-lang/sdk#24949 BUG=24949 [email protected] Review URL: https://codereview.chromium.org//1446383002 .
dart-lang · Nov 16, 2015 · 74e5478 · 74e5478
1 parent 83453a4
commit 74e5478
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+# 0.1.7
+
+* Fix to `package_api_docs` (dart-lang/sdk#24947; #154).
+
 # 0.1.6
 
 * Fix to `package_prefixed_library_names` (dart-lang/sdk#24947; #154).

diff --git a/lib/src/rules/package_api_docs.dart b/lib/src/rules/package_api_docs.dart
@@ -88,6 +88,11 @@ class Visitor extends GeneralizingAstVisitor {
   DartProject get project => rule.project;
 
   void check(Declaration node) {
+    // If no project info is set, bail early.
+    // https://github.com/dart-lang/linter/issues/154
+    if (project == null) {
+      return;
+    }
     if (project.isApi(node.element)) {
       if (node.documentationComment == null) {
         rule.reportLint(getNodeToAnnotate(node));

diff --git a/pubspec.yaml b/pubspec.yaml
@@ -1,5 +1,5 @@
 name: linter
-version: 0.1.6
+version: 0.1.7
 author: Dart Team <[email protected]>
 description: Style linter for Dart.
 homepage: https://github.com/dart-lang/linter