1. 15 6月, 2019 3 次提交
  2. 14 6月, 2019 1 次提交
  3. 08 6月, 2019 1 次提交
  4. 30 4月, 2019 1 次提交
    • H
      changed the way we report live analysis to task center (#35336) · 8d860e12
      Heejae Chang 提交于
      * changed the way we report live analysis to task center
      
      previously, we listen to diagnostic service to report progress. problem is that, it only raise event if it found errors on a file. so what we report is actually last file we found errors on rather than file that we are analyzing.
      
      this caused confusion since we report in task center that we are analyzing file "A" when it is actually "analyzed" not "analyzing"
      
      another issue is since it only report file that contains errors. we might not actually show anything in task center if there is no error, or show file "A" for long time if that is only file with errors.
      
      this PR changes the experience closer to what users would expects. and now progress is for solution crawler not specifically on diagnostics.
      
      now we report file that solution crawler is analyzing.
      
      there is still cavet such as solution cralwer can pause between processing a file if VS is busy. but it will still show file "A". or we will not update UI at least 200ms a part and etc.
      
      since it is task center where we don't want to be too impactful to VS, based on feeedback we will see whether we need to do more such as detect solution crawlwer pause and update task center to show pasue. or update task center to show different stage such as analyzing/analyzed.
      
      or show in task center, what analyzer is actually running such as diagnostic, todo, designer attribute scan, find all reference cache and etc.
      
      * addressing PR feedbacks
      8d860e12
  5. 19 4月, 2019 1 次提交
  6. 17 4月, 2019 2 次提交
  7. 27 3月, 2019 1 次提交
  8. 26 3月, 2019 1 次提交
  9. 23 3月, 2019 1 次提交
  10. 22 3月, 2019 1 次提交
  11. 20 3月, 2019 1 次提交
  12. 19 3月, 2019 3 次提交
    • A
      Prepare for UI review · f17f969a
      Andrew Hall (METAL) 提交于
      f17f969a
    • A
      UI updates post review · cf0f5890
      Andrew Hall (METAL) 提交于
      cf0f5890
    • A
      Implement MoveToNamespace feature · 2b8be03c
      Andrew Hall (METAL) 提交于
      Move to namespace allows a user to move all symbols in a context
      to a different namespace, whether it's a new one or existing. This is
      different from renaming a namespace as not all symbols in the namespace
      are moved, only ones within the current document.
      
      It relies on the IChangeNamespaceService implementation to do the heavy
      lifting. Since the current implementation only supports C#, this new
      feature will as well.
      
      After this is checked in, when a user has the cursor inside a namespace
      declaration that has no nested namespace declarations within it, a code
      fix will be suggested to pop up a dialog and help the user choose a new
      namespace.
      2b8be03c
  13. 28 2月, 2019 3 次提交
  14. 01 2月, 2019 3 次提交
  15. 04 1月, 2019 1 次提交
  16. 18 12月, 2018 1 次提交
  17. 11 12月, 2018 1 次提交
  18. 10 12月, 2018 1 次提交
  19. 07 11月, 2018 1 次提交
  20. 31 10月, 2018 1 次提交
  21. 23 10月, 2018 1 次提交
  22. 27 9月, 2018 1 次提交
    • M
      Add support for displaying reference kind to Find All References window. · 9ee0805a
      Manish Vasani 提交于
      Implementation of the custom column support has been kept extensible, so that each reference item can have customized key-values info, where key corresponds to the name of the custom dynamic column and values are set of one or more values the table entry. Columns become dynamically visible if at least one row in the table has a value for it.
      
      Fixes #24877
      9ee0805a
  23. 21 9月, 2018 1 次提交
  24. 20 9月, 2018 1 次提交
    • M
      Add support to save current tools options into an .editorconfig file · 588f1798
      Manish Vasani 提交于
      This ports https://github.com/dotnet/roslyn/pull/28472, that went into the `editorconfig-ide` feature branch, into `dev16.0.x` branch with following additional refactorings/enhancements:
      
      1. Generated .editorconfig file has all localized string as comments.
      2. No hardcoding of individual options in the editorconfig file generator. Instead, each `EditorConfigStorageLocation` now requires to provide a delegate to convert a given option value into the .editorconfig string representation (basically complimentary of the parsing delegate that was already required).
      3. Introduce concept of `OptionGroup` to enable grouping options in the same feature (such as CodeStyleOptions, CSharpCodeStyleOptions, FormattingOptions, CSharpFormattingOptions, etc.) into sub-features or groups (such as "this. preferences", "expression-level preferences", etc.). This enables the editorconfig generation to group the related options, without explicit knowledge of each option group.
      4. Expose `AllOptions` from each option feature (such as CodeStyleOptions, CSharpCodeStyleOptions, FormattingOptions, CSharpFormattingOptions, etc.), so that the corresponding `IOptionProvider` always stays in sync with implemented options.
      588f1798
  25. 16 9月, 2018 1 次提交
  26. 29 8月, 2018 1 次提交
  27. 28 8月, 2018 1 次提交
    • M
      Address PR Feedback: · a6db5441
      Manish Vasani 提交于
      1. Break the analyzer into two separate diagnostics for flagging unused and unread members. Latter rule has no code fix.
      2. Remove all UI option elements for the added diagnostics and turn the rules off by default for now.
      3. Bail out for invalid operations and syntax errors.
      4. Handle doc comments and nameof - report unread member diagostics instead of unused member diagnostics if referenced in non-read/write context.
      a6db5441
  28. 25 8月, 2018 2 次提交
    • M
      Address PR feedback · c1c54cc9
      Manish Vasani 提交于
      c1c54cc9
    • M
      Add analyzer/fixer to flag and remove unused private members (fields/methods/properties/events) · 76f4c480
      Manish Vasani 提交于
      Analyzer flags two cases: members with no read/writes and members with only writes.
      
      1. Members with no read or writes: `Type '{0}' has an unused private member '{1}' which can be removed.`
      2. Only writes: `Type '{0}' has a private member '{1}' which can be removed as the value assigned to it is never used.`
      
      Code fix removes the unused member declaration.
      Fixes #24225
      
      Open questions:
      1. Current analyzer design uses a single code style option for all members and both the above kinds of unused members. We can potentially have multiple options, but this should probably be done based on feedback.
      2. Should the analyzer use different diagnostic IDs for the above two kinds of unused members? This will mean that the FixAll experience will need multiple iterations for removing unused members.
      3. Should we update the code fix (or have an additional code fix) that also updates the write references for the case (2) above? Or is it better to leave the references to break the code so the user can analyze if the value being assigned can also be removed. The PR current chooses the latter approach as it is more conservative and unlikely to cause silent breaks.
      76f4c480
  29. 15 8月, 2018 1 次提交
  30. 21 7月, 2018 1 次提交