Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add two patterns to C# AutoCompleteRegexes #5285

Merged

Conversation

drewnoakes
Copy link
Member

This adds auto-complete/dictionary entries for the commit message text box based on new and using patterns.

As an example, in a recent commit message:

Use ArgumentBuilder and move SmartFormat reference

Before this change, both ArgumentBuilder and SmartFormat were flagged as spelling mistakes. These symbols were present in code resembling new ArgumentBuilder and using SmartFormat;.

Has been tested on (remove any that don't apply):

  • GIT 2.18
  • Windows 10

This adds auto-complete/dictionary entries for the commit message
text box based on `new` and `using` patterns.

As an example, in a recent commit message:

> Use ArgumentBuilder and move SmartFormat reference

Before this change, both ArgumentBuilder and SmartFormat were flagged
as spelling mistakes. These symbols were present in code resembling
`new ArgumentBuilder` and `using SmartFormat;`.
@codecov
Copy link

codecov bot commented Aug 4, 2018

Codecov Report

Merging #5285 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5285      +/-   ##
==========================================
+ Coverage   36.03%   36.03%   +<.01%     
==========================================
  Files         590      590              
  Lines       45256    45256              
  Branches     6224     6224              
==========================================
+ Hits        16309    16310       +1     
+ Misses      28196    28195       -1     
  Partials      751      751

@RussKie
Copy link
Member

RussKie commented Aug 4, 2018

We will need to write some serious tests around those arcane regex's

@RussKie RussKie added this to the 3.00 milestone Aug 4, 2018
@RussKie RussKie merged commit e1ac439 into gitextensions:master Aug 5, 2018
@drewnoakes drewnoakes deleted the csharp-auto-complete-patterns branch August 5, 2018 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants