VBA Rubberducking

Duga

12:01 AM

RELOAD!

[rubberduck-vba/Rubberduck] 7 issue comments.

[Zomis/Games] 3 commits. 2 closed issues. 23 additions. 36 deletions.

[Minesweeper] New Users: 21, Games Played: 200, Bombs Used: 129, Moves Performed: 27521

Duga

4:29 AM

[rubberduck-vba/Rubberduck] IvenBach synchronized pull request #5357: Issue5346 extract interface creates private classes

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5357: Issue5346 extract interface creates private classes

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=h1) Report
> Merging [#5357](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.03%`.
> The diff coverage is `21.43%`.

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

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5357: Issue5346 extract interface creates private classes

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=h1) Report
> Merging [#5357](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.03%`.
> The diff coverage is `21.43%`.

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

Duga

4:47 AM

[rubberduck-vba/Rubberduck] build for commit f1e41e66 on unknown branch: AppVeyor build succeeded

[rubberduck-vba/Rubberduck] build for commit f1e41e66 on unknown branch: 61.11% (target 0.00%)

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5357: Issue5346 extract interface creates private classes

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=h1) Report
> Merging [#5357](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.03%`.
> The diff coverage is `21.43%`.

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

[rubberduck-vba/Rubberduck] IvenBach synchronized pull request #5357: Issue5346 extract interface creates private classes

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5357: Issue5346 extract interface creates private classes

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=h1) Report
> Merging [#5357](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.03%`.
> The diff coverage is `21.43%`.

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

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5357: Issue5346 extract interface creates private classes

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=h1) Report
> Merging [#5357](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.03%`.
> The diff coverage is `21.43%`.

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

Duga

5:03 AM

[rubberduck-vba/Rubberduck] build for commit 864167e9 on unknown branch: AppVeyor build succeeded

[rubberduck-vba/Rubberduck] build for commit 864167e9 on unknown branch: 61.13% (target 0.00%)

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5357: Issue5346 extract interface creates private classes

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=h1) Report
> Merging [#5357](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5357?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.02%`.
> The diff coverage is `33.33%`.

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

Duga

1:32 PM

[rubberduck-vba/Rubberduck] MDoerner synchronized pull request #5497: Annotate Declaration Command

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5497: Annotate Declaration Command

> # [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/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.11%`.
> The diff coverage is `52.49%`.

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

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5497: Annotate Declaration Command

> # [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/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.11%`.
> The diff coverage is `52.49%`.

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

Duga

1:46 PM

[rubberduck-vba/Rubberduck] build for commit 5189907d on unknown branch: AppVeyor build succeeded

[rubberduck-vba/Rubberduck] build for commit 5189907d on unknown branch: 61.03% (target 0.00%)

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5497: Annotate Declaration Command

> # [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/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.11%`.
> The diff coverage is `52.74%`.

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

Duga

3:11 PM

[rubberduck-vba/Rubberduck] MDoerner synchronized pull request #5497: Annotate Declaration Command

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5497: Annotate Declaration Command

> # [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/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.11%`.
> The diff coverage is `52.74%`.

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

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5497: Annotate Declaration Command

> # [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/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.11%`.
> The diff coverage is `52.74%`.

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

Duga

3:26 PM

[rubberduck-vba/Rubberduck] build for commit 5ac6d388 on unknown branch: AppVeyor build succeeded

[rubberduck-vba/Rubberduck] build for commit 5ac6d388 on unknown branch: 61.03% (target 0.00%)

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5497: Annotate Declaration Command

> # [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/3fb577017cc2d4411f856bbb8eb2dfc2d3570df7&el=desc) will **decrease** coverage by `0.11%`.
> The diff coverage is `52.74%`.

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

Duga

3:38 PM

[rubberduck-vba/Rubberduck] BZngr synchronized pull request #5502: AssignmentNotUsedInspection reduce false positives due to jump statements

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5502: AssignmentNotUsedInspection reduce false positives due to jump statements

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5502?src=pr&el=h1) Report
> Merging [#5502](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5502?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/65bf91efc6f24fb872c6f09cea7bcdec017d846c&el=desc) will **increase** coverage by `0.04%`.
> The diff coverage is `86.67%`.

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

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5502: AssignmentNotUsedInspection reduce false positives due to jump statements

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5502?src=pr&el=h1) Report
> Merging [#5502](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5502?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/65bf91efc6f24fb872c6f09cea7bcdec017d846c&el=desc) will **increase** coverage by `0.04%`.
> The diff coverage is `86.67%`.

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

[rubberduck-vba/Rubberduck] MDoerner created comment on pull request #5497: Annotate Declaration Command

> I have moved the Add and Remove buttons as suggested and the result looks a lot better.
![AnnotateDeclaration_v2](https://user-images.githubusercontent.com/22868956/82757621-e85f8d00-9de1-11ea-8879-1c0712c3b898.jpg)

In addition, the inspection names are now sorted by localized name. I had to do this in the view model since WPF does not support sorting by converted items in a collection view. This means that a change in the UI culture is not reflected immediately in the sort order. However,

[rubberduck-vba/Rubberduck] MDoerner ready_for_review pull request #5497: Annotate Declaration Command

Duga

3:53 PM

[rubberduck-vba/Rubberduck] build for commit c8e95431 on unknown branch: AppVeyor build succeeded

[rubberduck-vba/Rubberduck] build for commit c8e95431 on unknown branch: 61.20% (target 0.00%)

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5502: AssignmentNotUsedInspection reduce false positives due to jump statements

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5502?src=pr&el=h1) Report
> Merging [#5502](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5502?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/65bf91efc6f24fb872c6f09cea7bcdec017d846c&el=desc) will **increase** coverage by `0.06%`.
> The diff coverage is `100.00%`.

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

[rubberduck-vba/Rubberduck] BZngr commented on Rubberduck.CodeAnalysis/Inspections/Concrete/AssignmentNotUsedInspection.cs of pull request #5502: AssignmentNotUsedInspection reduce false positives due to jump statements

Duga

4:09 PM

[rubberduck-vba/Rubberduck] Vogel612 commented on Rubberduck.Refactorings/AnnotateDeclaration/AnnotateDeclarationModel.cs of pull request #5497: Annotate Declaration Command

Duga

5:28 PM

[rubberduck203/VBEX] bhavikbansal starred us

M.Doerner

6:06 PM

@Duga Suggestions regarding the code pane context menu entry are welcome as well.

Duga

8:05 PM

[rubberduck-vba/Rubberduck] BZngr synchronized pull request #5410: MoveMember Refactoring

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5410: MoveMember Refactoring

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5410?src=pr&el=h1) Report
> Merging [#5410](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5410?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/65bf91efc6f24fb872c6f09cea7bcdec017d846c&el=desc) will **increase** coverage by `0.79%`.
> The diff coverage is `77.94%`.

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

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5410: MoveMember Refactoring

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5410?src=pr&el=h1) Report
> Merging [#5410](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5410?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/65bf91efc6f24fb872c6f09cea7bcdec017d846c&el=desc) will **increase** coverage by `0.79%`.
> The diff coverage is `77.94%`.

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

Duga

8:20 PM

[rubberduck-vba/Rubberduck] build for commit c8ef102d on unknown branch: AppVeyor build succeeded

[rubberduck-vba/Rubberduck] build for commit c8ef102d on unknown branch: 61.92% (target 0.00%)

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5410: MoveMember Refactoring

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5410?src=pr&el=h1) Report
> Merging [#5410](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5410?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/c8ef102d585948f56e0815e4d3efeed7ca51fe63&el=desc) will **not change** coverage.
> The diff coverage is `n/a`.

```diff
@@ Coverage Diff @@
## next #5410 +/- ##
=======================================
Covera

Transcript for