VBA Rubberducking

12:00 AM

BTW, which files had the merge conflicts? For some reason, your commit message does not contain that info.

RELOAD!

[rubberduck-vba/Rubberduck] 7 commits. 1 opened issue. 1 closed issue. 6 issue comments. 132 additions. 132 deletions.

[Minesweeper] Games Played: 111, Bombs Used: 86, Moves Performed: 15541, New Users: 12

[rubberduck-vba/Rubberduck] IvenBach synchronized pull request #5409: Add access keys for settings window

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5409: Add access keys for settings window

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

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

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5409: Add access keys for settings window

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

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

IvenBach

I'm positive there wasn't any changes. I did have one more addition.

Duga

12:17 AM

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

IvenBach

@Duga Now if I get a breadcrumb for stackoverflow.com/questions/62805519/… to figure out the access key issue I'll be able to remove the WIP label.

Mathieu Guindon

1:20 AM

@IvenBach thinking the textbox might need to be exposed as a dependency property

IvenBach

I tried that and couldn't get it to work.

That's not on the question?

:derp: I'll have to edit it after I'm back from shopping.

Duga

1:38 AM

[rubberduck-vba/Rubberduck] nelix starred us

3

IvenBach

2:15 AM

@Duga Wonder when Tuvok will give us a star.

2

IvenBach

3:13 AM

@IvenBach Iven-from-a-few-moments-not-too-long-ago showed that if you want to progress in knowledge you have to open your mouth and expose your own stupidity to start learnin' properly.

Duga

11:30 AM

[rubberduck-vba/Rubberduck] Irubataru created comment on issue #5315: AttributeValueOutOfSyncInspection should change attribute value by default

> As the fix all behaviour was changed in #5531 I am going to close this as it is no longer relevant.

[rubberduck-vba/Rubberduck] Irubataru closed issue #5315: AttributeValueOutOfSyncInspection should change attribute value by default

IvenBach

5:19 PM

I can't get RD to load anymore.

Working on resolving the access key issue and when compiling opening the VBA IDE there was no splash screen. Navigating to Add-ins menu and opening it that way says RD can't be loaded.

I've gone back to a previously stable build and still can't get RD to load.

Checking out next and building still has the issue. Nothing created in the logs directory either.

Checking out breaking feature branch and adding everything back magically has it work without issue.

I just don't understand...

Duga

5:42 PM

[rubberduck-vba/Rubberduck] IvenBach synchronized pull request #5409: Add access keys for settings window

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5409: Add access keys for settings window

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

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

[rubberduck-vba/Rubberduck] IvenBach removed label PR-Status: WIP from pull request #5409: Add access keys for settings window

[rubberduck-vba/Rubberduck] codecov[bot] edited comment on pull request #5409: Add access keys for settings window

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

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

Mathieu Guindon

@IvenBach usually that's a borked COM registration due to both dev and release builds coexisting

IvenBach

I haven't had a release build for a while.

Mathieu Guindon

hm

@IvenBach no problem then?

Duga

6:02 PM

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

IvenBach

6:12 PM