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

Issue 2946313003: Restore partial analysis of analysis options files (Closed)

Created:
3 years, 6 months ago by Brian Wilkerson
Modified:
3 years, 5 months ago
Reviewers:
devoncarew, scheglov
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+100 lines, -27 lines) Patch
M pkg/analysis_server/analysis_options.yaml View 1 chunk +1 line, -1 line 1 comment Download
M pkg/analysis_server/lib/src/analysis_server.dart View 1 chunk +4 lines, -0 lines 0 comments Download
M pkg/analysis_server/lib/src/context_manager.dart View 8 chunks +41 lines, -0 lines 0 comments Download
M pkg/analysis_server/lib/src/domain_completion.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analysis_server/lib/src/status/diagnostics.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analysis_server/test/context_manager_test.dart View 5 chunks +11 lines, -2 lines 0 comments Download
M pkg/analysis_server/test/domain_diagnostic_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analysis_server/test/integration/search/find_top_level_declarations_test.dart View 1 chunk +9 lines, -5 lines 0 comments Download
M pkg/analysis_server/test/src/plugin/plugin_manager_test.dart View 2 chunks +10 lines, -0 lines 0 comments Download
M pkg/analyzer/analysis_options.yaml View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer/lib/src/summary/package_bundle_reader.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer/lib/src/task/options.dart View 3 chunks +13 lines, -8 lines 0 comments Download
M pkg/analyzer_cli/analysis_options.yaml View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer_cli/lib/src/analyzer_impl.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer_cli/tool/perf.dart View 3 chunks +3 lines, -3 lines 0 comments Download
M pkg/analyzer_plugin/analysis_options.yaml View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (2 generated)
Brian Wilkerson
This causes analysis server to analyze analysis options files when the context root containing them ...
3 years, 6 months ago (2017-06-21 20:50:34 UTC) #2
devoncarew
lgtm Great to see this re-enabled :) I expect we'll see a lot of fixes ...
3 years, 6 months ago (2017-06-22 14:49:22 UTC) #3
Brian Wilkerson
Committed patchset #1 (id:1) manually as e28064f4050267c77a55d257010baf8198e7a2bd (presubmit successful).
3 years, 6 months ago (2017-06-22 15:00:57 UTC) #5
scheglov
3 years, 5 months ago (2017-06-26 15:58:36 UTC) #6
Message was sent while issue was closed.
LGTM

Powered by Google App Engine
This is Rietveld 408576698