« first day (2193 days earlier)      last day (27 days later) » 

12:00 AM
RELOAD!
[rubberduck-vba/Rubberduck] 3 issue comments.
[Minesweeper] Games Played: 105, Bombs Used: 69, Moves Performed: 14814, New Users: 6
 
1:16 AM
Initial implementation

Introduces ICodeBuilder and CodeBuilder in place of extension methods.
[rubberduck-vba/Rubberduck] BZngr pushed commit 98503e29 to next: Merge branch 'rubberduck-vba/next' into RefactorExtensionMethods_5419
[rubberduck-vba/Rubberduck] BZngr pushed commit 14abeaf5 to next: Cleanup from merge
[rubberduck-vba/Rubberduck] BZngr pushed commit dda811c1 to next: Merge branch 'rubberduck-vba/next' into RefactorExtensionMethods_5419
[rubberduck-vba/Rubberduck] BZngr pushed commit 8c6c97e3 to next: Merge branch 'rubberduck-vba/next' into RefactorExtensionMethods_5419
Merge pull request #5452 from BZngr/RefactorExtensionMethods_5419

Replace code generation extension methods with ICodeBuilder
 
SO ads - runner-up:
> Clicks per day: 7.48 (688 total)
ducky:
> Clicks per day: 12.37 (1,150 total)
that's just O__O crazy!
 
2:09 AM
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=h1) Report
> Merging [#5497](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/cfe1ad3820accea5347ae51d71068b735281ce4c&el=desc) will **decrease** coverage by `0.04%`.
> The diff coverage is `54.59%`.


```diff
@@ Coverage Diff @@
## next #5497 +/- ##
==============================
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=h1) Report
> Merging [#5497](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/cfe1ad3820accea5347ae51d71068b735281ce4c&el=desc) will **decrease** coverage by `0.04%`.
> The diff coverage is `54.59%`.


```diff
@@ Coverage Diff @@
## next #5497 +/- ##
==============================
 
2:23 AM
[rubberduck-vba/Rubberduck] build for commit e973f644 on unknown branch: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] build for commit e973f644 on unknown branch: 61.06% (target 0.00%)
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=h1) Report
> Merging [#5497](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/cfe1ad3820accea5347ae51d71068b735281ce4c&el=desc) will **decrease** coverage by `0.06%`.
> The diff coverage is `54.59%`.


```diff
@@ Coverage Diff @@
## next #5497 +/- ##
==============================
 
2:38 AM
[rubberduck-vba/Rubberduck] BZngr pushed commit d3368537 to next: Merge remote-tracking branch 'refs/remotes/rubberduck-vba/next' into next
Resolved issue

Added MCVE tests. Corrected XMLDoc example.
[rubberduck-vba/Rubberduck] BZngr pushed commit ed912ed7 to next: Addressed comments. Added more examples
[rubberduck-vba/Rubberduck] BZngr pushed commit 6fb89b7b to next: Merge branch 'rubberduck-vba/next' into 5490_FalsePositive
Improved XMLDoc example for Public std module procs

Identified example module as 'Macros' and added comment to reinforce why
Public procedures of Standard Modules are ignored by the inspection.
[rubberduck-vba/Rubberduck] BZngr pushed commit 2cfbb9d8 to next: Merge branch 'rubberduck-vba/next' into 5490_FalsePositive
Fix XMLDoc

Missing VBA Comment apostrophe
Merge pull request #5495 from BZngr/5490_FalsePositive

ProcedureNotUsedInspection false positive for PropertyLet
[rubberduck-vba/Rubberduck] build for commit 69bfff3f on next: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] build for commit 69bfff3f on next: 61.12% (target 0.00%)
 
 
8 hours later…
11:20 AM
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=h1) Report
> Merging [#5497](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/69bfff3f0756f56edc0c88667f19e1a9cd76731d&el=desc) will **decrease** coverage by `0.06%`.
> The diff coverage is `54.59%`.


```diff
@@ Coverage Diff @@
## next #5497 +/- ##
==============================
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=h1) Report
> Merging [#5497](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/69bfff3f0756f56edc0c88667f19e1a9cd76731d&el=desc) will **decrease** coverage by `0.06%`.
> The diff coverage is `54.59%`.


```diff
@@ Coverage Diff @@
## next #5497 +/- ##
==============================
> I have added some margin to the right of the bottom grid. I guess I forgot to add that after moving the buttons from its right to its top.

![AnnotateDeclaration_v3](https://user-images.githubusercontent.com/22868956/83967466-e9082100-a8c1-11ea-8e46-3dc65598b174.jpg)
 
11:34 AM
[rubberduck-vba/Rubberduck] build for commit 5f73f570 on unknown branch: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] build for commit 5f73f570 on unknown branch: 61.06% (target 0.00%)
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=h1) Report
> Merging [#5497](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5497?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/69bfff3f0756f56edc0c88667f19e1a9cd76731d&el=desc) will **decrease** coverage by `0.06%`.
> The diff coverage is `54.59%`.


```diff
@@ Coverage Diff @@
## next #5497 +/- ##
==============================
 
11:53 AM
> Closes #3745 This PR introduces the RenameFolderRefactoringAction, which allows to rename the final subfolder part of a given folder. It then replaces this part and retains both the parent folder and the subfolder structure---you can introduce intermediate folders, though. This is used to add a new RenameFolderCommand for the CE, which is now integrated into the RenameCommand to allow folder renames in the CE....
!RenameFolder Note that the capture has been made before fixing the order of folder and the folder name in the arguments to the instruction text. That is the right way around, now. To better split the functionality, a ChangeFolderRefactoringAction has been extracted from MoveFolderRefactoringAction, which allows to exchange the full folder while leaving the subfolder structure intact....
This is used both in MoveFolderRefactoringAction and RenameFolderRefactoringAction. This PR is still a draft because it is based on PR #5497. Once that PR is merged, I will rebase this one and remove the draft status. Then, it will be more clear in the diff what is actually new in this PR.
 
12:06 PM
> Closing as a duplicate of #5052.

It just seems that the recent fixes to the UI, which fixed a complete freeze of the VBE experienced by some users, made this problem worse again.
[rubberduck-vba/Rubberduck] build for commit c220b58c on unknown branch: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] build for commit c220b58c on unknown branch: 60.97% (target 0.00%)
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5515?src=pr&el=h1) Report
> Merging [#5515](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5515?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/69bfff3f0756f56edc0c88667f19e1a9cd76731d&el=desc) will **decrease** coverage by `0.15%`.
> The diff coverage is `51.39%`.


```diff
@@ Coverage Diff @@
## next #5515 +/- ##
==============================
 
 
2 hours later…
2:03 PM
> This adds a delay of 400ms to the search boxes in the test explorer, the inspection results window and the CE. This means that a somewhat below average typer with a typing speed of 150cpm will just not trigger a UI refresh while typing.

In particular in the CE, the delay is considerably shorter than the refresh time for the filter change.
> As the dupe above shows, this issue can lead to insane test runtimes.
We really should have a look into how to stop the UI from updating so much when we move items between groupings.

The root cause seems to be that we refresh the entire collection view each time a test completes.
https://github.com/rubberduck-vba/Rubberduck/blob/69bfff3f0756f56edc0c88667f19e1a9cd76731d/Rubberduck.Core/UI/UnitTesting/TestExplorerViewModel.cs#L233-L241

Since .Net 4.5, there is a a feature specifically
 
2:18 PM
[rubberduck-vba/Rubberduck] build for commit 3236197a on unknown branch: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] build for commit 3236197a on unknown branch: 61.12% (target 0.00%)
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5516?src=pr&el=h1) Report
> Merging [#5516](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5516?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/69bfff3f0756f56edc0c88667f19e1a9cd76731d&el=desc) will **decrease** coverage by `0.00%`.
> The diff coverage is `n/a`.


```diff
@@ Coverage Diff @@
## next #5516 +/- ##
=================================
 
3:18 PM
> I have tested with the refresh of the groupings deactivated entirely. That caused a runtime of 83s, which is worse than the live grouping. So, something else must be going on here.
> This uses live grouping and live filtering in the test explorer, which is the built-in functionality to allow updates of the grouping based on property changes. This is faster than refreshing the entire collection view each time and also does not have the negative side effects of changing the selected items and expended states all the time.

As such, it will help with #5052, but the main issue there seems to a different one.
 
3:36 PM
[rubberduck-vba/Rubberduck] build for commit 5ad1e116 on unknown branch: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] build for commit 5ad1e116 on unknown branch: 61.12% (target 0.00%)
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5517?src=pr&el=h1) Report
> Merging [#5517](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5517?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/69bfff3f0756f56edc0c88667f19e1a9cd76731d&el=desc) will **decrease** coverage by `0.00%`.
> The diff coverage is `100.00%`.


```diff
@@ Coverage Diff @@
## next #5517 +/- ##
=============================
 
neeevermind I'm an idiot
 
5:02 PM
 
 
1 hour later…
6:06 PM
[rubberduck-vba/Rubberduck] MDoerner pushed commit 5ad1e116 to next: Use live grouping and live filtering in TestExplorer
Merge pull request #5517 from MDoerner/UseLiveGroupingInTestExplorer

Use live grouping and live filtering in TestExplorer
 
6:53 PM
> I was doing some tidying and found this comment:

'@Ignore StopKeyword: For Debugging, once RD understands code paths this Ignore won't be necessary

It got me thinking, RD needs some way of telling me when an Ignore annotation is no longer necessary

**Description:**
A new meta-inspection perhaps, that flags `@Ignore[Module]` statements which no longer have any effect - perhaps because the code has changed or because RD's capabilities have improved and you no longer need to @Igonor
> If this is implemented as another inspection result, it would be assumed that it too could be ignored - if this is done on a case by case basis then it needs to annotate an '@Ignore comment/argument, not a module or statement.
 
7:18 PM
Add delay to search box input

This adds a delay of 400ms to the search boxes in the test explorer, the inspection results window and the CE. This means that a somewhat below average typer with a typing speed of 150cpm will just not trigger a UI refresh while typing.

In particular in the CE, the delay is considerably shorter than the refresh time for the filter change.
Merge pull request #5516 from MDoerner/DelayForSearchInput

Add delay to search box input
 
7:32 PM
> I think this would need to be a "Rubberduck Opportunities" meta-inspection - IIRC there's a flag for non-annotatable inspection results (currently only *DefaultProjectName* uses it) that could be handy here.

Basically the inspection would be flagging `@Ignore` and `@IgnoreModule` annotations that aren't muting any inspection results.... and doing that would need to change how RD tracks inspection results; ignored results are simply filtered-out/discarded... and no inspection currently has a
[rubberduck-vba/Rubberduck] build for commit 14acc3f4 on next: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] build for commit 14acc3f4 on next: 61.12% (target 0.00%)
[rubberduck-vba/Rubberduck] retailcoder pushed 27 commits to next (only showing some of them below)
Make Rename dialog use the TextBoxErrorStyle

This adds error tooltips.
In addistion, renaming to the same name is allowed now.
Enhance conflicting name message on rename

It now stated the full name of the conflicting declaration.
Ask for confirmation in MoveToFolder if folder will be merged

Also makes the dialog use TextBoxErrorStyle
Make the MoveToFolder dialog use the TextBoxErrorStyle

This allows error tooltips.
[rubberduck-vba/Rubberduck] MDoerner pushed commit 9e68b4da to next: Make individual empty subfolder names illegal in move (to) folder dialogs
[rubberduck-vba/Rubberduck] MDoerner pushed commit a80cf791 to next: Merge branch 'next' into AnnotateCommand
[rubberduck-vba/Rubberduck] MDoerner pushed commit e973f644 to next: Merge branch 'next' into AnnotateCommand
[rubberduck-vba/Rubberduck] MDoerner pushed commit c6214c92 to next: Merge branch 'next' into AnnotateCommand
[rubberduck-vba/Rubberduck] MDoerner pushed commit 5f73f570 to next: Add right margin to arguments grid of AnnotateDeclaration UI
Merge pull request #5497 from MDoerner/AnnotateCommand

Annotate Declaration Command
 
 
3 hours later…
10:34 PM
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5515?src=pr&el=h1) Report
> Merging [#5515](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5515?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/7accdd2a5c577fcf364c169fae952d056a256768&el=desc) will **decrease** coverage by `0.09%`.
> The diff coverage is `41.31%`.


```diff
@@ Coverage Diff @@
## next #5515 +/- ##
==============================
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5515?src=pr&el=h1) Report
> Merging [#5515](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5515?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/7accdd2a5c577fcf364c169fae952d056a256768&el=desc) will **decrease** coverage by `0.09%`.
> The diff coverage is `41.31%`.


```diff
@@ Coverage Diff @@
## next #5515 +/- ##
==============================
 
10:49 PM
[rubberduck-vba/Rubberduck] build for commit 53e2b494 on unknown branch: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] build for commit 53e2b494 on unknown branch: 60.96% (target 0.00%)
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5515?src=pr&el=h1) Report
> Merging [#5515](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5515?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/7accdd2a5c577fcf364c169fae952d056a256768&el=desc) will **decrease** coverage by `0.09%`.
> The diff coverage is `41.31%`.


```diff
@@ Coverage Diff @@
## next #5515 +/- ##
==============================
 

« first day (2193 days earlier)      last day (27 days later) »