« first day (2062 days earlier)      last day (1118 days later) » 
00:00 - 14:0014:00 - 00:00

2:00 PM
I was talking about trying it to one of my coworkers, and she said she got out of it because her income was being taxed at about 40% because she was paying company and personal taxes on the same money.
 
all of our bonuses and the sort at the current company are just lumped into regular paychecks to be taxed; good/bad/indifferent, that's how it is
 
Whereas our tax rate as just employees is closer to 20%, or so.
 
company tax?
i take it that's a state thing?
 
Not sure exactly how it came out; I didn't look into it.
Basically, though, she was paying nearly twice as much tax as a self-employed contractor than as an employee.
Plus she had to cover all her own benefits.
 
ahh; benefits is a big part
 
2:02 PM
@Cyril I wish that were how it was with us. Yeah, it's a state tax, but if they roll it into your paycheck it's just taxed at the regular rate. Ours is a separate check, so that's "separate income" or whatever. At least I live in Delaware so there's no sales tax. Double taxation, am I right?
 
lol, DE has its own little problems, but that does make sense. i take it you're at an LLC which makes more than $1M in revenue... they have lots of weird things about that. it's why i see so many companies "doing business" in DE and NJ, despite being in Cali or something
er, less than* not more than
i think that's the cutoff? it's beena long time since i've had to deal with negotiating law-language in contracts... something about company has to be taxed but if the company doesn't make enough it's from employee paychecks otherwise it's a lesser rate for the company if they do make above the threshold
 
I'm in the pharma industry, we're all in DE
 
as am i... the industry, not in DE, though i think our contractual terms are about to swap back from DE to NJ due to the recent change in required LOL and import/export language for DE businesses
 
2:18 PM
@M.Doerner so IIUC, I basically need to add another condition to the multiple OR statements in the IsIgnoredDeclaration() in the inspection's definition, right? let me try that and see what happens
 
so, having not tried it, how is RD shaping up in support of VBA-Word?
 
I'm actually waiting on IT to approve the RD request, hopefully it goes through. I'm very excited to try it out, VBA-Word is so much messier than Excel, I'm confident anything will help lol
 
@jclasley are you not able to install on yourself? It can be installed as per-user and thus not require admins.
 
i think it would overall be helpful because it adds the more IDE functionality
 
@jclasley welcome to the pond!
 
2:26 PM
@this I didn't consider that, I totally should've done that but now that i've sent the request it's on their radar... I'll have to debate with the angel and devil on my shoulder
 
Nah, at least you're a good citizen.
I only asked because I was curious if they would have prevented installation even on per-user.
 
They might. They're hesitant to allow anything because of the nature of our work and regulatory bodies.
 
2:52 PM
@M.Doerner apparently those annotations don't get sent to the declaration for whatever reason. for all of the declarations that appeared, the last one is the only one that isn't ignored for whatever reason
i'll throw more time on it later
 
LOL, just tried to initiate the download myself via github, the download AWS page was blocked through our web filter with the category "Meaningless Content". That's a bit harsh
 
GitHub runs on AWS?
I wonder if they'll switch it to Azure sometime?
 
@jclasley lolwat?! Try the AppVeyor build link; under the "assets" tab you'll find the installer executable AppVeyor uploads to GitHub
@Hosch250 odds: likely
 
@MathieuGuindon we're in business!
 
woohoo
 
3:08 PM
Yay!
 
Anyone care to share a screenshot of their preferred layouts?
 
3:27 PM
?
 
Like organization of windows and panes in your VBE.
 
I like having Code Explorer (with properties underneath) to the right, immediate pane at the bottom; I usually float other toolwindows, or dock them at the bottom
 
Likewise; except that I dock my code explorer & properties to the left.
I don't bother docking other toolwindows to the bottom. Too cluttered and clumsy given VBE's crapatastic docking system.
 
if i need immediate window i pull itup with ctrl+g, but typically leave it off of my screen
 
needs tabs.... AvalonMention++
 
3:38 PM
everything else is docked left, except that button toolbar for userforms which floats on the main monitor and i have to pull it over
 
Man y’all would hate mine. I have watches pinned right (though I’m suspecting I was using it to do a lot of things RD will do for me), explorer pinned left, immediate pinned bottom. I’m currently trying to figure out unit tests, this is all very new and very exciting for me given I have no actual programming background.
 
Were there tabs, I would definitely have the unit test explorer docked in the same space with the immediate.
 
3:56 PM
0
Q: Checks data already stored in specific worksheets then populates data as needed

Zack EThe code below looks at different range values and then populates specific cells on a worksheet called GeneralInfo in the Worksheet_Activate event.The below code does work as expected, but its time to clean it up a little bit, so I am looking for some ideas. I have tried to split up the code wit...

 
Just read an article on how not to make a law.
 
Middle monitor has Project and Properties on the left: 1-3 modules in the middle then Immediate/Locals on right. Test and Code explorer float on left monitor over the Word instance, Right monitor is other stuff.
 
The #fun part was postscript 2. Apparently certain laws the Nazi's made were still considered in effect after the war, but decisions based on said laws were overturned because the judges misinterpreted their own laws.
 
4:12 PM
So I'm going through and adding descriptions to my modules and procedures, but I don't see them popping up. For modules, I'm adding it right at the top, just below Option Explicit, am I doing it incorrectly?
 
4:41 PM
@jcrizk Can you post the additional condition you added?
 
sure, let me fire up VS
apparently my profs don't want to give me homework yet lol
This is the additional condition
|| declaration.Annotations.Where(x => x is ITestAnnotation).Any();
This is the context
ProcedureNotUsedInspection.IsIgnoredDeclaration()

        {
            var enumerable = classes as IList<Declaration> ?? classes.ToList();
            var result = !ProcedureTypes.Contains(declaration.DeclarationType)
                || declaration.References.Any(r => !r.IsAssignment && !r.ParentScoping.Equals(declaration)) // recursive calls don't count
                || handlers.Contains(declaration)
                || IsPublicModuleMember(modules, declaration)
                || IsClassLifeCycleHandler(enumerable, declaration)
 
@jcrizk You compile your code? I kick it out the door letting it fend for itself. "You're a big boy now."
 
@IvenBach as long as i say it works, then it works
 
In the land of rainbow sparkles and unicorn candicorn droppings everything is delish.
 
@jcrizk You might want to reread what I wrote earlier. Then, you might see the problem.
In addition, you do not need the Where; the lambda can go into the Any.
 
4:55 PM
@jclasley Welcome to the pond.
 
@IvenBach thanks man
 
@M.Doerner yeah i definitely misread that when i woke up lol but what you said makes sense to me
yep that did it
thanks lol it was so damn simple, but i think going through the debugger still helped me understand a bit more about how it works
 
5:12 PM
@jclasley We've all mostly figured things out as we've gone along. You're in good company. Ask if you have any questions.
@jclasley I add module annotations above Option Explicit. That ensures they are part of the module declaration.
 
[rubberduck-vba/Rubberduck] jonathan-rizk pushed commit 8781f560 to InspectionsIgnoreTestAnnotations: ProcedureNotUsedInspection ignores declarations that have ITestAnnotation annotations.
 
@jcrizk Did you just push to Rubberduck's main repository?
 
i branched off of next and made a different branch
im writing a PR for the changes i made so you guys can rip me a new one ;)
 
You should fork the repo, push to your own fork and then do a pull request.
I'll go and delete that branch.
 
5:23 PM
why should i do that? just curious
i've no clue what the advantage of doing that is
 
should i clone my fork, then? or is it all the same if i just work in the clone of the RD repo itself
my guess is that there's some magic i don't know about
 
@jcrizk you don't have to re-clone, but you'll want to re-wire up your remote to point to your fork
 
ok sweet
i figured i could do that, but i wasn't sure
 
that's git remote something something github url something, hold on...
5897
A: How to change the URI (URL) for a remote Git repository?

hobbsYou can git remote set-url origin new.git.url/here (see git help remote) or you can just edit .git/config and change the URLs there. You're not in any danger of losing history unless you do something very silly (and if you're worried, just make a copy of your repo, since your repo is your hist...

 
5:35 PM
why did they think that Sign Off was no longer needed in a remote desktop service is beyond me...
 
@MathieuGuindon thanks
 
Now that I am at an actual keyboard, let me explain some advantages of a fork workflow.
Most of them are advantages for the main repo and other contributers.
For one, nobody has to do the maintenance to delete no longer required branches on the main repo.
In addition, nobody will fetch not necessary feature branches when fetching from the main repo.
Moreover, only stuff verified to be good via a PR process will ever end up in the main repo.
 
I understand it as cleanliness.
 
those are some pretty good points lol
 
The advantage for the contributer is that he can push to his fork whatever he wants. Nobody will complain.
 
5:42 PM
We don't see each others FUBAR attempts midway during a PR.
 
those resolve some of the huge issues i have with people who are not as organized as i am
read as OCD
 
In particular, partial work can be pushed to save it.
 
5:57 PM
alright, i think i did it correctly
 
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5367?src=pr&el=h1) Report
> :exclamation: No coverage uploaded for pull request head (`InspectionsIgnoreTestAnnotations@29ce25a`). [Click here to learn what that means](https://docs.codecov.io/docs/error-reference#section-missing-head-commit).
> The diff coverage is `n/a`.
 
Interesting, there do not seem to be notifications for branch deletions.
 
lemme check the webhook settings
apparently I have forgotten to reinstate that setting after my tag-deletion spree
 
The mention of marker interface got me wondering and I found an interesting article that suggests that marker interfaces wants to be a visitor.
 
6:16 PM
[rubberduck-vba/Rubberduck] build for commit 29ce25a7 on unknown branch: AppVeyor build succeeded
 
@this i thought of another implementation lol just add a default property to annotation base called something like IsTestAnnotation and then default it to false for everything but the ones that we want to identify as test annotations
 
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5367?src=pr&el=h1) Report
> Merging [#5367](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5367?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/0c4bb18c0fbb147f0bafe3933dc70a557eb72777?src=pr&el=desc) will **increase** coverage by `0.01%`.
> The diff coverage is `100%`.


```diff
@@ Coverage Diff @@
## next #5367 +/- ##
=========================
[rubberduck-vba/Rubberduck] build for commit 29ce25a7 on unknown branch: 100% of diff hit (target 60%)
[rubberduck-vba/Rubberduck] build for commit 29ce25a7 on unknown branch: 61.67% (target 0%)
 
@jcrizk I do not know enough about the annotations to comment; Max might have more insight. I recall Vogel did express that he was unhappy with how the test annotations were structured few months ago. Whether the visitor pattern is the answer, I don't know.
 
6:31 PM
:twitch: WHAI?! What purpose does docs.microsoft.com/en-us/office/vba/api/excel.font.fontstyle serve? There's already Font.Bold and Font.Italic property.
 
@IvenBach Apparently none, seeing as it doesn't exist.
NVM, I found it after fixing your markdown :)
 
^ That
 
I'm having a :derp: day for hypelinks somehow.
 
I is hyped about your hypelinks.
 
Style is a little more powerful.
As you see, you can set both at the same time, and there are likely other values you can set too.
 
6:34 PM
it might be also a later addition or a legacy property
 
> Acceptable values are Regular, Italic, Bold, and Bold Italic.
 
Hmmm.
Probably just is the driving property. The Bold and Italic flags probably just modify it.
 
Once exposure has occurred it cannot unoccur.
 
i'm hoping that it was the original property before they realized their mistake and added the Bold and Italic properties
but I don't know the history
 
This pond fundamentally changed how I think about code.
 
6:36 PM
Eh, there is still a bit of that kind of thing in HTML.
 
then they realized what a big mistake it was.
 
You can set lots of things with one property or each individually.
 
I shudder to think about the pre-CSS formats
 
Like: border: 1px solid black.
Or border-color: black, etc.
Ehhh, it can't be any worse than it is now...
I mean, now they have a lot more stuff done wrong on top of that.
 
the problem is more that it makes it much more difficult to lint or enfroce style
 
6:39 PM
You think that's goofy? In Word VBA you can infinitely refer to a table and the range of the table without ever losing any properties. Selection.Tables(1).Range.Tables(1).Range.Tables(1).Range.Tables(1).Range.Tables(1).Select
 
And CSS is awful because it cascades and it's a pain to create custom components.
 
@Hosch250 that's easy to fix. Add !important everywhere.
 
Oh, and don't forget that each browser can display each control however they want with whatever defaults they want.
@this That doesn't really help.
 
@Hosch250 Note it was served with a dash of sarcasm.
@jclasley I think that's just circular reference
There are lot of things like that in various OM.
 
Yeah, IIRC, Excel Sheet stuff has that.
It's basically a parent/child relationship.
 
6:41 PM
I suppose, but you can traverse the structure as well, because it's not contained. If any part of the table is within the range, the whole table is in the range.
 
?Application.Application.Application.Application.Application.Application Is Application
 
That's the whole point.
 
@jclasley it would have made it much more problematic if they represented the table in a selection differently than the table itself, I suspect.
 
I mean you can traverse and then come back in the same reference, if that makes sense... Word VBA ranges are just weird
 
Word ranges make perfect sense to me. But then again I don't do Excel. The big mistake with Word is to assume that a keyword after a range is a narrowing action.
 
6:47 PM
LOL, I did not think I'd see that in a modern application:
The trash can to right says Run garbage collector
 
It doesn't look very modern...
Looks like it's from mid 00s.
 
it actually was updated fairly recent
last dec, it looks like
 
The eager beaver is ready to eat up all your RAM.
 
^
 
@Freeflow yeah I mean it took me a few weeks to understand that ranges are spatial representations anchored at two points and encompass all the objects therein, but that’s mainly because those were the first two weeks I learned a single thing about programming
 
6:59 PM
maybe I should pit Rubberduck with Beaver.
 
I feel like MrExcel blatantly chooses not to improve his coding abilities.
His utter disregard for Option Explicit... :breathe:
 
Really?
 
Keep in mind the type of person who would choose to name themselves “MrExcel” lol
 
I'm not surprised
 
That code example has several wrongs in it.
 
7:03 PM
not able to deduct from the article alone that he doesn't use Option Explicit
it might only have the Sub, without the entire module?
 
Blech, shorthand Range syntax
 
No he disregards Option Explicit.
@this msg isn't declared anywhere...
 
I guess he must be that good!
 
Nobody is that good. Not even you or Mug.
 
To code up something flawlessly....
 
7:04 PM
@this The situation described in the article you linked is completely different.
Here, we use the marker interface as a substitute for what would have been a flag when we were still using an enum.
 
@M.Doerner Cool. It was only really tangential since I got to wondering exactly why marker was a code smell
 
Public msg as String
 
@jcrizk I took the liberty to put the magic incantation into your PR description that will automatically close the issue once your PR gets merged.
 
7:14 PM
@MathieuGuindon Do we still need the branch ac-defaultsettings-patch? It has been merged already.
 
Any good resources for learning C#?
 
I think what is a good resource very much depends on which way you learn best.
 
Failure is my #1 learning method.
 
Definitely not a fan of following tutorials, probably just documentation and problem-solving.=
 
Trying something I somewhat understand leads me down many rabbit holes.
 
7:19 PM
Many people like the hands on approach of the book from the Head First series.
However, I personally hate that series.
 
^^ I benefited from it.
 
I would also not recommend the method I learned C# to anybody else.
 
Okay, I won't ask then.
 
I simply read most of C# in a Nutshell.
 
It was hand-holdey to get me past the crawling stage. After I understood enough C# Head First did feel lacking.
 
7:22 PM
What about Big Nerd Ranch? I've been using their book for Swift and am liking it so far.
They don't have a C# book, so never mind on that.
 
Might be worth a try. I personally never looked at that series.
Oh, OK.
 
@M.Doerner hm I always forget to delete the branch for some reason... feel free to delete it, I can't see how on mobile
 
I personally despise series that's basically a "... For Dummies" clone because even if you knew nothing, the value is ruined once you've known just enough --- if I'm buying a book, I want it to have "re-readability" value
For that reason, I'm more attracted due books that are more reference-oriented than "click this to do y"
cookbooks are also a good example since they can provide some samples for one to then generalize but I can't really recommend that to total beginner because there's not enough background to make any good generalization.
 
Unless the potential coder is willing to devote time and effort to it no book will help.
 
7:54 PM
Duck check: Should With fooWorksheet.Range("A1") come up as an inspection for default member access? All properties accessed within the with block are qualified and this feels off to me. I can't remember if I'd asked before, apologies if so.
 
8:09 PM
Mug. Have you thought about writing an article on how to Not use On Error Resume Next as a crutch, and instead write code properly that avoids it, when not needed?
 
@IvenBach I don't understand what you mean, On Error Resume Next is super useful in quite a few instances. The one that comes to mind is returning only unique values by adding to a collection and skipping those that throw an error.
 
@M.Doerner An ally!
@jclasley MS Docs are pretty good for C#.
 
[rubberduck-vba/Rubberduck] MDoerner deleted branch ac-defaultsettings-patch
 
8:24 PM
@Hosch250 I'll give that a try.
 
@jclasley if you really want to stretch your thinking have a bash a Rust......
 
or Haskell
 
TBH I think Rust is easier than Haskell
It's still fundamentally a procedural language.
whereas Haskell is a different class
 
@jclasley indeed, when used properly ;-)
 
@jclasley Don't you want a hashset for that?
OK, I got some Docker stuff working together using Docker Compose... Looks like somehow they are getting on the same network this way. I'll try this for my other stuff tomorrow, I guess.
I'm not good at sales calls.
Was just involved in one.
 
8:40 PM
@jclasley I found "Learn C# in one day" by Jamie Chan was great for me. I had no experience with any language except VBA so it was perfect for me.
 
@Hosch250 man look at you talking to me like I know what a hashset is hahaha
 
@jclasley just beware of the VBA for professionals type book which runs to over 1000 pages and yet never goes near an interface.
 
@SmileyFtW I will take a look! Thanks.
@Freeflow I've never used a VBA book, only ever read the documentation, so I'm leaning toward using documentation for C# as well. The only problem is the language surrounding VBA is inherently simpler, so I'm thinking the C# docs will be.... challenging
 
@jclasley Practically a list of unique items. Technically, it's more than that, but practically, that's it.
It can have performance benefits too, as access is O(1) for a specific item. That isn't a guarantee for all use cases.
 
@jclasley Dictionary.Exists()? Or am I missing something?
 
8:50 PM
He may be referring to the error catching function for collections, not dictionaries
 
Ah yes, that's what I missed, he specifically said collections.
 
Thanks @Cyril!
 
@puzzlepiece87 It'll still generate an error if you try and add something with the same key, which is kind of the whole point.
 
rather than triggering an error, you can use an inherent function to dictionaries to say "if not dic.exists(Key) then dic(key) = foo". different approaches
basically build as to not cause an error; that can oftentimes be seen as "lazy", but in several cases it is completely appropriate to use/trap an error
 
8:59 PM
Public Sub OldFoo()
    Dim filteredWorksheet As Worksheet
    Set filteredWorksheet = ActiveSheet

    Dim checkFilter As Filter
    Set checkFilter = filteredWorksheet.AutoFilter.Filters(2)

    Err.Clear
    On Error Resume Next  'This should be avoided. No reason for this at all.
    Dim itemCount As Long
    itemCount = UBound(checkFilter.Criteria1) 'Is .Criteria1 an array?
    On Error GoTo 0

    Dim itemString As String
    If Err.Number = 0 Then
        Dim counter As Long
        For counter = 1 To itemCount
@jclasley ^
There's absolutely no need for that On Error Resum Next (OERN) to be there.
If IsArray(checkFilter.Criteria1) Then
    itemString = Join(checkFilter.Criteria1, vbNullString)
Else
    itemString = checkFilter.Criteria1
End If
That accomplishes the same thing while being self documenting and much clearer.
 
I mean, I don't really do Excel VBA so I can't really speak to that specific situation, but that first bit of code seems overly-complicated, I agree.
But, for instance, I have an automator that opens up anywhere from 3-5 word documents in the background and parses information, depending on user input. Now of course I could add them to a collection when opening them and then loop through and close them, but I can also just On Error Resume Next and then close all of them regardless of if they're actually open or not
Is that bad form?
 
if you were required to use an On Error there, that code looks like a good usecase for On Error GoTo, but Iven's solution seems much cleaner
 
9:14 PM
@jclasley What happens when you close a document that's not open?
 
@Hosch250 Object not set error
 
Exactly. And why would you throw an error? Errors are for things you absolutely can't control.
That's like bringing in the B2s to clear a path from one part of town to another with bombs instead of just driving on the existing roads.
And it'll slow your application down, too, because it has to waste time processing the error.
 
@jclasley And I actually worked through the book in a day and felt like I had really gotten a good start.
 
Lots of C# programmers think this way too, between just blindly catching exceptions they could avoid and using them for control flow.
 
Exceptions are for exceptional circumstances. Things you can't recover from.
 
9:21 PM
And then there are the ones who are the opposite. "Don't use Single--it could throw an exception if multiple items get in the DB."
Instead of, I don't know, maybe checking that the write to that field in the DB had proper checks maybe?
And only writing to that DB table in one dedicated service so you could keep track of it?
 
9:34 PM
OERN: Same old same old. Just because you can do something doesn't mean that you should.
 
9:45 PM
According to MS docs a module name cannot exceed 255 characters. RD seems to limit the length to something shorter when selecting the module in CE and clicking Rename in the lower info panel. Is that by design?
 
the module name might refer to the fully qualified name?
if that's the case RD should limit to 255 - 1 - [project name]
 
no that's because we have dumb hosts
some hosts will enforce a lower limit
I think Excel sheets won't let you name it more than 64? 32? characters
there's a old closed GH issue about that.
 
Word has a 32 character limit
 
For Excel, the limit is 32 as well.
I regularly curse that.
 
31 characters fro module names.
 
9:57 PM
Ah yes, the doc I was looking at was not for Module names
sorry
@M.Doerner I am now... :)
 
31 chars is too restrictive; disemvowel if you must.
 
I hate the most that the VBE simply reverts whatever you entered to what was there before without giving you a chance to shorten whet you have already entered.
 
VBIDE: Sorry. You were 1 character too many. Say goodbye to your succint name.
MDoerner: You're tempting me to go get my coffee right now...
 
VBIDE Team in 90s: 31 characters should be more than enough.
 
10:41 PM
man, i thought of another way to implement the changes i made in the PR that i like better lol
why now, brain
 
What way did you think of?
 
i could just add a property called IsTestAnnotation to both AnnotationBase & IAnnotation, default it to false in the constructor, and then make it true only for the annotations that we want to decide are test annotations
 
I do not really like that option.
 
what are the drawbacks of it?
 
It adds stuff to an interface that only makes sense for a small number of implementations.
 
10:46 PM
that is definitely true
 
I kind of violates the interface segregation principle.
The inspection you changes is basically theonly consumer.
Other annotations are consumed by a lot more stuff, which has no need for the information whether the annotation is for tests or not.
 
i get what you're saying though
it's like unnecessarily inflating the amount of info that's exposed from all annotations
 
We do something similar on IdentifierReferences, but that is more lazyness than anything else.
Yep
 
you would definitely not like the stuff i need to use at work lol
i definitely dont
there are classes that have like 30 properties, but hardly any of them ever get used
 
I bet these properties are meant to be used by different consumers of the class, right?
 
10:51 PM
no
worse
 
BTW, we have a kind of similar situation with Declaration.
 
the issue is that developer 0 said "designing this to be easy to expand upon is not my concern", and so they implemented a circular hierarchy with almost every class hierarchy lol
and it doesn't even need to be that way
a lot of stuff has become way too highly coupled together
and unit testing is almost nearly impossible because of this, but i've been getting support from other devs in our project to communicate to management that untangling this mess is important because it will then allow us to more easily locate some of the huge issues that we have
i almost forgot about those unnecessary namespaces lol
 
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5367?src=pr&el=h1) Report
> Merging [#5367](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5367?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/0c4bb18c0fbb147f0bafe3933dc70a557eb72777?src=pr&el=desc) will **increase** coverage by `0.01%`.
> The diff coverage is `100%`.


```diff
@@ Coverage Diff @@
## next #5367 +/- ##
=========================
 
11:18 PM
[rubberduck-vba/Rubberduck] build for commit 57521e50 on unknown branch: AppVeyor build succeeded
> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5367?src=pr&el=h1) Report
> Merging [#5367](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/5367?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/0c4bb18c0fbb147f0bafe3933dc70a557eb72777?src=pr&el=desc) will **increase** coverage by `<.01%`.
> The diff coverage is `100%`.


```diff
@@ Coverage Diff @@
## next #5367 +/- ##
=========================
[rubberduck-vba/Rubberduck] build for commit 57521e50 on unknown branch: 100% of diff hit (target 60%)
[rubberduck-vba/Rubberduck] build for commit 57521e50 on unknown branch: 61.66% (target 0%)
 
00:00 - 14:0014:00 - 00:00

« first day (2062 days earlier)      last day (1118 days later) »