VBA Rubberducking

Duga

00:00

RELOAD!

[Cardshifter/Cardshifter] 42 commits. 6 opened issues. 1 closed issue. 12 issue comments.

3

[Cardshifter/cardshifter.github.io] 1 commit.

Mat's Mug

Holy c...cardshifter shafted us big time!

Duga

[retailcoder/Rubberduck] 5 commits. 3 opened issues. 10 closed issues. 27 issue comments.

Phrancis

@Mat'sMug We've been busy ;D

Mat's Mug

I see that!

Phrancis

s/shafter/shafted/

Mat's Mug

00:02

Phone

Phrancis

I'm mostly doing documentation, but @Simon has been furiously adding awesome new effects and features. Who knew JavaScript could do all this stuff?

Simon André Forsberg

@Phrancis Technically, Java could do them too

Phrancis

Right. Meant it as a compliment to your JavaScript skillz, not as a frown on Java at all :)

Mat's Mug

@SimonAndréForsberg hey there! I have a feature request for @Duga! :)

Simon André Forsberg

in TCG Creation, 1 min ago, by Duga

[Zomis/Duga] retailcoder opened issue #82: Include created tags in reload summary

Mat's Mug

00:09

Thoughts?

Simon André Forsberg

@Mat'sMug I like it approved

@Phrancis Want me to reimplement it in Brainfuck?

3

Mat's Mug

Yay!

Phrancis

@SimonAndréForsberg Please don't! ;p

Simon André Forsberg

@Phrancis No worries, I have a Sudoku Solver and a Battleship game to implement there first.

TTGTB

RubberDuck

Thanks for the tip btw @Phrancis.

@Mat'sMug maybe it's a good idea to undo this? help.github.com/articles/changing-access-permissions-for-wikis

RubberDuck

00:34

Nope nope nope. Staying out of the 2nd Monitor for now.

Phrancis

00:48

@RubberDuck Wise ducky.

RubberDuck

I have my moments.

Phrancis

So easy to write documentation for Simon's code

keywords.effects.damage = {
    /** Build action description */
    description: function(obj) {
        return "Deal " + valueDescription(obj.value) + " damage to " + obj.target;
    },
    /**
     * Deals damage to a target.
     * @param {Object} obj - The applicable card object along with related properties/values.
     */
    action: function (obj) {
        return function (me, event) {
            var target = entityLookup(me, obj.target);
            var value = valueLookup(me, obj.value);

Mat's Mug

01:18

2nd monitor is starting to gather a number of interesting quotes right now ;)

Marc-Andre

I knew tags where kind of a grey area in Code Review, but wow when someone wants to debate a point they can go on for some time. I think arguing about brackets is easier than that.

Mat's Mug

lol some monkeys are more ...tenacious than others ;)

Phrancis

02:42

@Marc-Andre And you thought tabs vs. spaces was tedious!

Mat's Mug

03:26

0

Q: Hitchin' a ride

Some background About 6 months ago an idea emerged in the VBA Code Review chatroom, and together with a fellow VBA reviewer and CR addict, I started working on what became Rubberduck, an open-source project that has seen quite a lot of its source code posted and peer reviewed on Code Review. I ...

Phrancis

03:50

@RubberDuck @Mat'sMug If you came upon this documentation, would you think it good enough? (note: it is not finished)

Mat's Mug

> noAttack: 1

shouldn't that be noAttack: true?

Phrancis

The Targets / Filters section is fuzzy right now, will get better after I talk to Simon

@Mat'sMug Maybe. It's JavaScript... 1 === true === "1" ;D

But would be wise to make a note that 1 and true do the exact same thing

Mat's Mug

even more so - the section starts by saying Boolean values are noted as true or false, but then examples have true as 1

Phrancis

^ +1

Mat's Mug

and this makes it more confusing:

> enchantment: true

Phrancis

03:57

OK good, I'll fix that and make it consistent. Anything else jump at you?

Mat's Mug

> Applies the contained effects

I'd write it as "applies the specified effects"

"Applies to the owner of the affected cards." -> specifies that an effect's target is the owner of the card (if I read this correctly)

> Applies to active owner.

headsplode

> Applies to inactive owner.

needs more clarification IMO.. but I know very very little about the project

Phrancis

Yeah, that part is fuzzy

12 mins ago, by Phrancis

The Targets / Filters section is fuzzy right now, will get better after I talk to Simon

Mat's Mug

other than that.. looks good :)

Phrancis

04:13

@Mat'sMug Awesome. We should have a snapshot release soon, hopefully get some CR folks to trial the game.

I'm actually really amazed by how far we've gone in the last week-or-so

Mat's Mug

4 hours ago, by Duga

[Cardshifter/Cardshifter] 42 commits. 6 opened issues. 1 closed issue. 12 issue comments.

well...

I think that's a record!

Phrancis

Agreed

in TCG Creation, Apr 29 at 19:45, by Simon André Forsberg

@skiwi Well, we need to decide on and implement something.

I think this^ gave all of us a bit of a kick

Mat's Mug

I'm out - 'night!

Phrancis

04:32

'night!

Hosch250

04:51

advancedinstaller.com

Found this while researching something completely unrelated.

in WPF on Stack Overflow Chat, Apr 13 at 17:55, by Reed Copsey

We've settled on http://www.advancedinstaller.com/ - which I highly recommend, especially for .NET projects

Duga

10:00

[retailcoder/Rubberduck] ckuhn203 pushed 7 commits to next

Duga

10:39

[retailcoder/Rubberduck] ckuhn203 pushed 186 commits to sourcecontrol

[retailcoder/Rubberduck] ckuhn203 pushed commit d973c144 to sourcecontrol: Synced SourceControl With Next

[retailcoder/Rubberduck] ckuhn203 created pull request #444: Added basic local functionality for Git Integration to merge sourcecontrol into next

[retailcoder/Rubberduck] ckuhn203 added label source-control-integration to pull request #444: Added basic local functionality for Git Integration

[retailcoder/Rubberduck] ckuhn203 added label settings-config to pull request #444: Added basic local functionality for Git Integration

RubberDuck

> I've also included some useful Mocks in the Unit Testing Project.

Duga

[retailcoder/Rubberduck] ckuhn203 merged pull request #444: Added basic local functionality for Git Integration from sourcecontrol into next

[retailcoder/Rubberduck] ckuhn203 pushed 52 commits to next

2

\[[**retailcoder/Rubberduck**](https://github.com/retailcoder/Rubberduck)\] [**ckuhn203**](https://github.com/ckuhn203) pushed commit [**6a4e63bf**](https://github.com/retailcoder/Rubberduck/commit/6a4e63bffe74257cfe90af899484c1be5d80fb95) to [**next**](https://github.com/retailcoder/Rubberduck/tree/next): Merge pull request #444 from retailcoder/sourcecontrol

Added basic local functionality for Git Integration

Mat's Mug

11:36

Is nobody else spotting the irony here? A meta post about the project for yet another way of discovering it. Genius! — Joe 13 mins ago

Oh wow! SCI hits next release!!

RubberDuck

12:01

@Mat'sMug Yes. If all goes well. I think so.

RubberDuck

12:12

That's a very interesting question. It must be possible, because Auto-cad did it, but I have no idea how. — RubberDuck 10 secs ago

FreeMan

@Mat'sMug You're a good mug. Great question to ask.

@Phrancis That is some exceptionally not finished documentation! All I see is "404". :(

FreeMan

12:48

@Phrancis If this is your first time playing Cardshifter, you may want to have a look at the Game Rules. I think my daughter will like your game - she like to make up the rules as she goes along! :)

RubberDuck

Mornin' freeman.

FreeMan

'sup?

RubberDuck

Not much. More data entry. Almost done I think.

Nothing like eating your own dog food though. This tool I built sucks.

I've got some serious upgrades planned for it.

FreeMan

13:15

@RubberDuck Automotive engineers should be required to spend time maintaining the vehicles they design. Architects should be required to move furniture in and out of houses they design. Programmers should be required to use the interfaces they've written...

2

RubberDuck

@FreeMan pinned

FreeMan

:)

FreeMan

13:38

flagged for migration to CR

RubberDuck

Good call @FreeMan. Looks good to my eyes.

(and I need another two performance upvotes for a shiny new badge)

RubberDuck

14:02

Holy shit I just figured out how to reference the VBA runtime.

FreeMan

What is the Certainty Level in @Duga s playground?

Mat's Mug

@RubberDuck hug

RubberDuck

Don't get too excited yet. Doesn't seem to much of anything useful in here...

Add Reference > Browse > C:\Windows\SysWow64\msvbvm60.dll

You have to browse for it though. Selecting "Visual Basic for Applications" under COM won't allow you to add it.

Phrancis

14:34

Monking

Mat's Mug

hi!

FreeMan

yo!

Phrancis

@FreeMan Sorry, in between posting the link and later last night, I ended up moving the documentation to its own folder here

@FreeMan That's great, now she can even make her whole complete game (and learn a little JavaScript while she is at it ;)

Not that the "Targets / Filters" section is pretty fuzzy right now, I need to have that clarified by the dev who wrote the code

RubberDuck

16:06

And they're dissolving my department. This is either a really good thing, or a really bad thing.

Mat's Mug

hmm

best of luck!

RubberDuck

Thanks buddy.

Duga

[retailcoder/Rubberduck] retailcoder assigned retailcoder to issue #445: Extract Method doesn't always work with valid selections

[retailcoder/Rubberduck] retailcoder added label bug to issue #445: Extract Method doesn't always work with valid selections

[retailcoder/Rubberduck] retailcoder opened issue #445: Extract Method doesn't always work with valid selections

> This will probably just magically go away with the upcoming refactoring/rewriting of the feature to leverage the `Declarations` API, but just so it's documented...

Making a valid selection (i.e. within a single method) sometimes seems to swallow an exception and *do nothing*.

Repro:

Public Function Create(adoRecordset As ADODB.Recordset) As SqlResult

Dim result As New SqlResult

'' SELECTION START ***********************************

[retailcoder/Rubberduck] retailcoder added label refactorings to issue #445: Extract Method doesn't always work with valid selections

FreeMan

16:23

@RubberDuck A potential upside: You may not have to deal with

3 hours ago, by RubberDuck

Nothing like eating your own dog food though. This tool I built sucks.

RubberDuck

Not an upside. I need to fix it before things change.

It's a matter of professional pride.

FreeMan

^valid point

Phrancis

18:18

If SSRS would stop timing out on me, I might be able to get some work done today.

RubberDuck

What would you want that for?

Phrancis

Not me, my boss ;)

Phrancis

18:37

GTFO. That report is still running. Good grief.

Phrancis

18:51

@RubberDuck @Mat'sMug I thought you'd find this response funny ^^

Mat's Mug

talk about talkative!

RubberDuck

I did. Thanks for that @Phrancis.

I'l lurking in the 2nd Monitor if anyone wants to chat.

FreeMan

@Phrancis simple enough...

Phrancis