Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(680)

Issue 2502233004: report errors in included options files (Closed)

Created:
4 years, 1 month ago by danrubel
Modified:
4 years, 1 month ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

report errors in included options files Example error messages include: /analysis_options.yaml(10..27): The include file other_options.yaml in /analysis_options.yaml cannot be found. /analysis_options.yaml(10..27): Bad options file format (expected String scope key, got YamlScalar) in /other_options.yaml(0..0) /analysis_options.yaml(10..27): Warning in the included options file /other_options.yaml(47..49): The option 'ftw' isn't supported by 'errors'. Since the included options file may not be part of the project, the error is reported on the initial include statement and includes the needed location information for the user to track down the problem. R=brianwilkerson@google.com Committed: https://github.com/dart-lang/sdk/commit/0d15f1d892afebc7bc1186857f3eeb585406b692

Patch Set 1 #

Patch Set 2 : merge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+191 lines, -10 lines) Patch
M pkg/analyzer/lib/source/analysis_options_provider.dart View 4 chunks +10 lines, -1 line 0 comments Download
M pkg/analyzer/lib/src/analysis_options/error/option_codes.dart View 2 chunks +37 lines, -0 lines 0 comments Download
M pkg/analyzer/lib/src/task/options.dart View 4 chunks +73 lines, -9 lines 0 comments Download
M pkg/analyzer/test/src/task/options_test.dart View 1 chunk +71 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
danrubel
4 years, 1 month ago (2016-11-16 23:07:46 UTC) #2
Brian Wilkerson
lgtm
4 years, 1 month ago (2016-11-17 00:46:50 UTC) #3
danrubel
4 years, 1 month ago (2016-11-17 19:43:11 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
0d15f1d892afebc7bc1186857f3eeb585406b692 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698