Yes! And I now have a full stack trace, so I might be able to finally fix this!

Thank you M$ Microsoft. VSC is the best thing ever.

awesome!

        foreach (VBComponent component in this.vbe.ActiveVBProject.VBComponents)

That doesn't work if the IDE isn't open yet ^^

correct, the addin isn't loaded until the IDE loads it :)

@RubberDuck wait till you try ReSharper!

It worked when I built it because I was loading and unloading the add-in from the IDE.

Okay @ducky.. now how do we fix it?

Check to see if the add in is loaded... extract the create logic into a method and call it on the button click instead. If it doesn't already exist.

Okay @ducky. That should work.

does it?

Yes. It works. Pull request will be incoming soon.

at last!

:)

what was it?

It was this @Mat'sMug.

oh

I had to change it so it initializes the task list after the IDE is fully loaded.

Just submitted the pull request. Fast forward merge. =)

Test Explorer is dockable, but I'm going to add an issue about decoupling the test engine from the menu.

good idea

merged. now resolving local conflicts :)

    public TestMenu(VBE vbe, AddIn addInInstance):base(vbe, addInInstance)
    {
        TestExplorerWindow testExplorer = new TestExplorerWindow();
        toolWindow = CreateToolWindow("Test Explorer", testExplorer);
        _engine = new TestEngine(vbe, testExplorer, toolWindow);
    }

I don't like that. Not sure how to fix it yet.

oh, [&Test] menu is back on its own

On purpose, or by accident?

I must have went a bit too fast with conflict resolution

took remote copy

didn't want too much trouble

;)

I missed something in the merge anyway. I'll take it.

so.. I think every custom menu item should go under the "Rubberduck" menu

and.. hold on, I'll put a comment on your issue there

Yeah. Probably. I think there's a case to add the ToDo list under view in addition to Rubberduck though.

or perhaps we should have a [Windows] menu under [Rubberduck]?

Look under view now

but yeah, maybe

lol

I didn't even know it was there

hmmmmmm why is VS ignoring my .gitignore?

task list has nothing in it :) nevermind, you have to select Comments

Ohhhhh that's a good point.

R# is picking up 8 todo's, VS task list only 7

+bug comment

I don't get it. *.user is clearly excluded, but VS wants to add it......

could it be me that keeps putting it back in?

nope

I don't think so. It has a path to Excel 12. That's me.

> Since all custom menu commands would be under the "Rubberduck" menu, I think we should derive them from some CommandBase class. Ideally generic/abstract enough to work just as well for a menu or a toolbar - like, let's define the commands, and let the App determine where to put them (in the RubberduckMenu, and/or an eventual RubberduckToolbar).

I've extracted RubberduckMenu into its own file, pushing now

done, you should be able to sync without issues

back to my parser issues

I <3 R#

@RubberDuck I created+assigned an issue and linked to #38; for now I'm only changing enough to build - I don't want to touch the menus and resolve conflicts later ;)

R# is a freakin' mindreader!

I'm out Mug. Cya later.

Later!

@enderland this is a fantastic answer

that will be my quit message haha

@enderland hope you dont mind me quoting it in my ABOUT ME !!! hHAh

i just completely love that

> I'm gracing you with an answer to your question (or edits, comments, etc) because I want to - not because you need an answer.

Amen @enderland.

lol. thanks :)

It was a bit of frustration with playing janitor. :\

That's why I'm a big fan of CR. We don't get the sheer volume of crap.

I think it helps that we work with working code.

I might be posting on SO later though. I think I'm misunderstanding the this keyword in C#

this refers to the current instance, like Me in VB

I'm not in front of the code right now, but I'm having trouble understanding how

in the base class...

I'll figure it out.

@RubberDuck I think ignorance or "this works before so we're going to do this" is the answer - not a FP background

@RubberDuck any object can be null ;)

Both should be referring to the same reference...

Yeah, but they should both be pointing to the same variable.... they're not.

wth

I think it's an inheritance thing that I'm not doing right.

Will post a screenshot at lunchtime.

Ok

I want to help that poor soul, but I need full working code to do it....

Well I could go in and suggest a way-overkill Delegate class to do it ;)

^^ bingo

lol

You're boring me today @ticker.

lunch screenshot time?

I opted for a shower instead. Things are hectic with work and the busted up wife.

Really bad XY question.

meh, VTC'd

SMH...... You've got a whole team of lazy people eh?

@ticker, no JAVAScript....

Use JavaScript to write viruses...