« first day (1292 days earlier)      last day (1888 days later) » 

12:00 AM
That's usually better done in xaml, there must be a reason why it's in plain C# code
 
Yeah, I figured it needs properties in order to execute the sorting over, but it still has to be done w/ OrderBy(GroupingDescription).ThenBy(SortDescription)?
 
nope. The collectionview does its own sorting, using sortdescriptions
 
#TIL
 
RELOAD!
[rubberduck-vba/Rubberduck] 3 commits. 3 opened issues. 1 closed issue. 14 issue comments. 20 additions. 2169 deletions.
 
It's possibly doing that, but it's its job, not ours
 
12:01 AM
[Zomis/FactorioMods] 1 issue comment.
 
@Mat'sMug Good. Means I'm on the right track.
I'm confused because it comes up as a getter.
Don't have time right now. I'll just ask bad questions. I'll look at it later tonight when I can adequately devote the time needed.
 
12:21 AM
Ayyyyiiiieeeee this keeps getting cmplicated and complicated
I can't mock a static class. :(
 
No but you can wrap it with an object, and that object can implement any interface you need
 
yeah, i'm reading up on that.
 
Oh I just received my Hacktoberfest tee (at last!)
2
 
great!
 
12:59 AM
Duck check: When updating code that then breaks tests you need to update the test in the manner you expect?
 
> #3485 sorts on description. F5 tested and UnitTested.

Unit tests were reordered to match VM ordering.
 
Home Time.
 
[rubberduck-vba/Rubberduck] build for commit eea22686 on unknown branch: AppVeyor build succeeded
 
1:23 AM
@IvenBach tests should match the specs. If the specs change, the tests change too.
 
1:40 AM
My answer to What is so wrong with TDD? https://www.quora.com/What-is-so-wrong-with-TDD/answer/Robert-Martin-9?srid=397z
 
2:28 AM
[retailcoder/Rubberduck] retailcoder pushed 66 commits to ProjectRefs (only showing some of them below)
[retailcoder/Rubberduck] IvenBach pushed commit 86834c08 to ProjectRefs: Declutter/Revert expression body.
Merge pull request #3612 from retailcoder/next

Decommissioning SynchronizeAttributesQuickFix
Merge pull request #3562 from IvenBach/Rubberduck.UI_C#6&7_Update

C#6/7 syntax updates
Merge pull request #3573 from IvenBach/Rubberdeck.Inspections_C#6&7_Update

C#6/C#7 technical debt payment
Merge pull request #3583 from IvenBach/Rubberduck.RegexAssistant_C#6&7_Update

C#6/C#7 technical debt payment
Merge pull request #3609 from IvenBach/Rubberduck.SmartIndenter_C#6&7_Update

C#6/C#7 technical debt payment
Merge pull request #3606 from Vogel612/some-critical-bugs

Prevent silent crash at startup.
Merge pull request #3620 from Hosch250/FixResx

Attempt to fix Resx
[retailcoder/Rubberduck] retailcoder pushed commit 7d91da1a to ProjectRefs: removed doomed attributes/annotations inspections
 
@Mat'sMug got it
 
:+1:
 
oh shit
 
What?
 
2:43 AM
nothing
 
[retailcoder/Rubberduck] retailcoder pushed commit c9fd241a to ProjectRefs: removed dead/empty folder
Revert "removed inspections... ...again?"

This reverts commit 80a090a4522f8b3277446142038ded61c8eb0dec.
 
git savetheday
 
LOL.
 
Are you now pushing to your test folder any PR?
 
He does it for big ones, especially from new-ish contributors.
 
2:45 AM
My wording was all FUBAR'd. Thanks for understanding what I meant
 
[rubberduck-vba/Rubberduck] retailcoder pushed 29 commits to next (only showing some of them below)
[rubberduck-vba/Rubberduck] IvenBach pushed commit 19af1694 to next: Rubberduck.Settings
[rubberduck-vba/Rubberduck] IvenBach pushed commit 2fe14f61 to next: Rubberduck.UI\Settings
[rubberduck-vba/Rubberduck] IvenBach pushed commit 43b095db to next: Rubberduck
[rubberduck-vba/Rubberduck] IvenBach pushed commit 9fb1ca15 to next: Breaking changes?
[rubberduck-vba/Rubberduck] IvenBach pushed commit 8147beb7 to next: Fix erro introduced
[rubberduck-vba/Rubberduck] IvenBach pushed commit 2a21d5cf to next: Address @Vogel612's comments.
[rubberduck-vba/Rubberduck] IvenBach pushed commit 15fb0376 to next: Merge branch 'next' of github.com/rubberduck-vba/rubberduck into Rubberduck.TodoItems_C#6&7_Update
[rubberduck-vba/Rubberduck] IvenBach pushed commit e503678c to next: Merge branch 'next' of github.com/rubberduck-vba/rubberduck into Rubberduck.TodoItems_C#6&7_Update
[rubberduck-vba/Rubberduck] IvenBach pushed commit f4c970fe to next: Merge branch 'next' of github.com/rubberduck-vba/Rubberduck into Rubberduck.TodoItems_C#6&7_Update
 
****.
 
Merge pull request #3571 from IvenBach/Rubberduck.TodoItems_C#6&7_Update

Rubberduck c#6&7 update. Second half of Rubberduck folder
 
You caused conflicts with my work.
 
I swear it's not intentional...
 
2:47 AM
@IvenBach #3582 is still marked as changes requested
@Hosch250 by letting these PR's collect dust for too long
 
No.
Well, yess.
I renamed something.
 
Want me to sync it up with RD or too late?
Stuffing my face with food.
 
Too late. It's merged.
Nothing serious.
Resolving them now.
 
[rubberduck-vba/Rubberduck] build for commit 372fdbd0 on next: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] Imh0t3b pushed commit f9726388 to next: German translation Inspections
[rubberduck-vba/Rubberduck] Imh0t3b pushed commit 6b22a39c to next: German translation Inspections
Merge pull request #3613 from Imh0t3b/next

German translations for inspection resource strings.
 
@Hosch250 Sorry. Forgot to keep them syncd with RD.
 
@IvenBach It was just we both changed the same things.
 
There we go.
 
[rubberduck-vba/Rubberduck] build for commit 5582c1ca on unknown branch: AppVeyor build succeeded
 
> public Visibility ExportVisibility => CanExecuteExportAllCommand ? Visibility.Collapsed : Visibility.Visible;
So, why not use the BooleanToInvertedVisibilityConverter class?
 
3:02 AM
I haven't used converters before. Never got back to it to use it.
Or update it it use it, that is.
 
you just define a static resource for one
and then use it as a parameter to your binding
 
It's basically a bit of code I wrote that can be used in the XAML to convert your boolean to the visibility.
Well, that's the standard way.
You can also create an instance of one and use that instance :)
I don't remember where I saw how to do that, unfortunately.
I feel like I'm beginning to stagnate in my job.
I'm not constantly being challenged to learn new things.
 
well, you're already authorized to work on RD if you get bored, no?
 
I'm hoping that changes in a little bit--we are bringing RabbitMQ and a service bus architecture in to support things.
@Mat'sMug If they can't keep me busy, which they can.
 
You can always attribute the PR to them.
 
3:06 AM
We have so much code-punching to do, but not enough learning.
Also, I got assigned to create a RSS feed for part of our website, so that'll be new.
 
now that was a moronic comment
 
1
Q: How to control error handling and synchronization in Antlr 4 / c#

LioI'm using Antlr 4 with c# target. Here is a subset of my grammar: /* * Parser Rules */ text : term+ EOF; term : a1 a2 a3; a1: .... ... ... I want to accept valid data blocks as (term)s, when error exists I want to search for the next valid term and print out the whole text which caused the ...

LOL.
Holy cow:
> It's expected that the implementer is already an expert in ANTLR 4's implementation of the Adaptive LL(*) parsing algorithm (we're talking researcher-level understanding).
 
I missed something.
What does LL mean?
 
Not sure. Low Level? :P
Really, it's the ANTLR slow-parse.
SLL is the fast one.
 
3:12 AM
And that means closer to machine code than typing in an IDE?
SLL = ??
 
I have no idea.
 
Super Low Level?
 
> It parses the input from Left to right, performing Leftmost derivation of the sentence.
 
Thanks. I read the abstract and didn't understand what it was saying.
 
4
Q: Antlr Error Strategy to skip tokens until rule matches again

metacircleI tried this solution but it didn't seem to work for me Here's an excerpt from my grammer: module : BEGIN MODULE IDENT STRING module_element* END MODULE ; module_element : element_1 | element_2 | element_3 | ... ; There is a bigger tree below each element. Now when a Recognit...

Just sticking crap in here. Just maybe we'll be recovering from errors before I'm done with my vacation.
 
I need to find out where my Office install info is at so I can work on RD on my new machine.
I can now fail even faster than before!
 
[rubberduck-vba/Rubberduck] build for commit 2d2f4dbf on unknown branch: AppVeyor build succeeded
 
@Hosch250 From Wiki, FYI.
 
I've deleted some branches from my GH account. I still see them in VS even after I've done fetch. Is there a way to make sure what I have in GH is reflected in VS. Have it delete anything that is comitted and syncd?
@Hosch250 I had created a repo to ask about that github.com/rubberduck-vba/Rubberduck/compare/…
 
3:50 AM
> Well.... I was considering the scenario where the `FinalReleaseComObject` returns a nonzero result... That is IMO, a fatal error because its basically saying we can't release it and probably should not be at this moment. So an Error level seems more apropos there.

That said, we ought to *trace* the non-final calls to `ReleaseComObject`. That would greatly aid us in tracking whether non-final calls allow us to gently put down the referenced COM object in appropriate order. We shouldn't be rely
[rubberduck-vba/Rubberduck] build for commit 583c1b67 on unknown branch: AppVeyor build succeeded
[rubberduck-vba/Rubberduck] retailcoder pushed commit 7d91da1a to next: removed doomed attributes/annotations inspections
[rubberduck-vba/Rubberduck] retailcoder pushed commit dd08e413 to next: removed doomed quickfix
[rubberduck-vba/Rubberduck] retailcoder pushed commit 80a090a4 to next: removed inspections... ...again?
[rubberduck-vba/Rubberduck] retailcoder pushed commit c9fd241a to next: removed dead/empty folder
Revert "removed inspections... ...again?"

This reverts commit 80a090a4522f8b3277446142038ded61c8eb0dec.
Merge pull request #3623 from retailcoder/ProjectRefs

Removed the doomed attributes/annotations inspections and the synchronizing qickfix.
[rubberduck-vba/Rubberduck] build for commit de3de3f8 on next: AppVeyor build succeeded
 
4:22 AM
@IvenBach In your local or your remotes?
Because you could first try a pull, and second try just deleting them from there.
 
I had them in my remote and deleted them. I thought that a fetch would mark them as having been deleted from the repo, or something similar to that.
 
fetch doesn't alter your branch
AFAIK
did you pull?
 
fetch makes branches show up in the list of remotes.
 
I did Pull and Fetch both.
If a branch previously existed in both the remote and local but was deleted from remote is there a way to have that reflected? Currently I'm just doing manual deletion and re-checking my GH repo for those branches that were merged and subsequently deleted.
 
@Duga I'm hoping this doesn't bite me too bad. I think it's tough enough to require me to learn new things while not too hard that I'm incapable of completing it.
 
It's really easy.
You shouldn't have to learn new thing to solve that one :P
Just the structure of the code.
 
> I've been working on this in my own coding. Let me know once this documentation has been written.
 
5:12 AM
Bed time. Catching up on missed sleep
 
 
2 hours later…
8:16 AM
Hi folks! I've maybe discovered my future contribution to rubberduct :-) I don't know C# (for now, planning to learn a bit), BUT I do speak language that's not there yet. :)
What do you think. I can help with Czech translation.
 
 
3 hours later…
11:42 AM
@SonGokussj4 absolutely!!
 
@Mat'sMug Nice. So I found this page, I download this, install Java on system, somewhat start that and I would start translating it by this program?
 
@Mat'sMug, are you available a second?
I have a few questions about your article at rubberduckvba.wordpress.com/2017/10/25/userform1-show
 
 
I think, since we ought to release as many of our COM objects as feasible, I will move the release code back into the COM wrapper and remove the new interface before adding the logging. Any objections, @Mat'sMug @this?
 
If you do try that, at least make sure you trace the count. I doubt we did that before, and I think we do need that information to tell us if we are releasing properly
 
I already planned to add that.
 
:+1:
 
The question was just about making the Release method available on all safe COM wrappers.
Later, we might want to wire that into disposal, but I would refrain from that in a first iteration.
 
12:05 PM
yes, with the info we now know, it would help to trace them all and see what happens. I don't think we used that information previously and just assumed it didn't work.
@SonGokussj4 that was the old way - @Vogel612 wrote this tool before they came up with a better resx editor --- I think you want this: marketplace.visualstudio.com/…
 
I'll do the changes and push them to the PR when I am back from work later today.
 
@M.Doerner sounds good!
 
@this Oki. I've downloaded the ResX Resource Manager as standalone portable app. (Don't have visual studio)
 
@NelsonVides what's up?
 
12:14 PM
@Mat'sMug I'm not getting the MVP thing...
 
@SonGokussj4 that's awesome!
 
And found some language files here: github.com/rubberduck-vba/Rubberduck/tree/next/… but I don't think these are the right ones. :) Whad do I download and open to start editing?
 
I also don't get which class your various excerpts of code belong to
 
@NelsonVides I'll put together an example project that illustrates it and the article that followed
 
@NelsonVides FWIW, you have both MVP and MVVM for RD dialogs
I had updated the wiki a bit to help with that confusing two-layered pattern
@SonGokussj4 I believe you would create a new file --- copy from Rubberduck***.resx
and create a Rubberduck***.cz.resx, just how like there are RubberduckUI.fr.resx and RubberduckUI.de.resx, so you'd have a RubberduckUI.cz.resz
 
12:20 PM
@SonGokussj4 you'll still need to fork the repository and download a local clone
 
@this can't the resxmanager handle that though?
 
IDK. Never used it. :\
 
Pretty sure it does
 
You can use "Add language" in the RESX Manager
 
12:34 PM
@Mat'sMug Will be my first forking :)
Already tried some translations. It automatically created the .cs file
So do I have to make all translations before I can do the push request thing?
 
No, but usually that makes things easier
 
And is there a way I can try the translated text within RB withou push request and you building it?
 
Not that i know of, tbh
 
Not without building the solution
 
But there shouldn't be any problems with this. We'll need to add resource strings for the new language, though
There's more than only RubberduckUI that needs translations btw
 
12:39 PM
The <no project> thing is worrying me
Can the resx manager open the .sln file?
That way you'll see all the resource files in the solution
 
@Vogel612 Oh. Good. :) Well, I don't expect I would translate it within a week or two :-D Just in my free time. So I'll let you all know when I have a big chunk and you can implement it or tell me what else should I download and try to translate :-)
 
the inspections have a bunch of resources too
 
@Mat'sMug I don't think so. I put there a folder with (sln, resx files) and it only shows resx.
 
Hmm when you use it within VS it shows all the .resx files in the solution in that left-side panel
 
@Mat'sMug Ah, I only downloaded 3 files for test purposes. Didn't downloaded the whole Duckie yet locally.
 
12:45 PM
Ah that's why
 
Sorry. :)
For misunderstanding.
 
If you clone your fork, git will track your local changes and you can git commit and git push them
You'll have to do that to make a PR on github
 
I plan to do that at home. :) I'm at work right now.
 
standalone files are possible I guess, but that'll be harder / more manual
@SonGokussj4 cool :-)
 
@Mat'sMug I will learn how to do PR from some videos. :) Anyway, I will report here just before that so you will know. :)
 
12:50 PM
Tip: when you have a whole file done, git commit -m "added cs translations" - and then git push origin and then a bus can run your laptop over and you'll still be able to recover your work from anywhere =)
(assuming you weren't too near the bus/laptop interaction)
 
:D
@ thanks for tips. I know git even with commits, pushing to origin or my GitHub repo or GitLab, even deleting some big files from history and customizing my log view. I don't say I know whole lot of git but I know the basics.
 
awesome
 
It's the Push Request and Forking and doing it with other people which I didn't do till now.
 
The forking part is already done =)
 
And I don't use much branches :) Too little projects and not much people (only me) for that :-)
 
12:54 PM
Pull Request is just a github mechanism to take commits from a fork/branch into another fork/branch
If you don't make changes other than to .resx files, you probably don't need any branching
 
Yeah. Understand the idea, will try the implementing soon enough :) Anyway, I have to work now so see ya for now :)
 
Later!
TIL that under the covers, a JavaScript array is really a dictionary with integer keys this language is crazypants
lol
 
1:07 PM
^ waitaminute, Javascript's a language?!?
 
 
 
1 hour later…
3:17 PM
@SonGokussj4 Woohooo!
 
>
```
2017-12-19 10:00:55.2626;FATAL-2.1.1.2532;Rubberduck.Common.LogLevelHelper;
Rubberduck version 2.1.1.2532 loading:
Operating System: Microsoft Windows NT 6.1.7601 Service Pack 1 x64
Host Product: Microsoft Office 2013 x86
Host Version: 15.0.4911.1000
Host Executable: EXCEL.EXE;
2017-12-19 10:09:56.4411;TRACE-2.1.1.2532;Rubberduck.UI.SourceControl.SourceControlViewViewModel;Cloning repo;
2017-12-19 10:09:56.4461;TRACE-2.1.1.2532;Rubberduck.UI.SourceControl.SourceControlViewVi
> Actually, this might be the same as 3594, apologies if it is.
> yes, it is. @NelsonVides when will you push the fixed library configuration?
 
4:03 PM
@Mat'sMug Can I tell you something?
 
lol of course why ask
 
Our parsing construction is a bloody mess.
I have it recovering from errors.
And I added a hack so it would listen for errors and set the parser state to Error if there is an error.
The remaining problem is to make the window listen for errors too.
 
well, the whole paradigm will need to shift when we recover from parsing errors; we'll have to dump the "error" parser state
 
Yeah, but right now, we are using that to determine whether to display the error list.
 
correct
 
4:07 PM
I'll just keep adding hacks to make it work for now :P
 
whatever works :)
 
Oh, I see.
So, our errors are really just looking at the list of exceptions, which we don't have anymore.
 
we'll probably need to encapsulate them in some kind of model/viewmodel class
(which we already do, only ..too late in the game)
 
@Hosch250 is that the reason for #2918?
 
No.
This is completely on my local.
 
4:13 PM
> which we I don't have anymore.
we'll still need to somehow know where the parse errors are, so we need a way for each modulestate to hold some IList<ParseError>
(and somehow populate it)
 
@Mat'sMug Yeah, we have an error listener.
 
@Mat'sMug IDKIUB doubt it belongs in a viewmodel. Seems to me that it should have no UI concerns and therefore be its own thing.
 
right, whatever. model then
yeah, model
 
Sounds right to me
 
whatever, some ParseError class that encapsulates the location and message
 
4:20 PM
naw. Hosch, call it OhEverythingIsGoingToHellInAHandBasket be creative with your naming. Break up the monotony of RD's codebase, you know?
 
@Mat'sMug SyntaxErrorInfo already exists.
@this Why a hand basket?
 
@Hosch250 ah, perfect then
@Hosch250 yeah, better use a truck. then we can have truckloads of $#!% going to hell
 
IDK. It's what everyone said. Must be a reason for it. Maybe they have silly import laws when going there?
 
Talking about import/export laws...
It's illegal to cross the MN border with a duck on your head.
I guess politicians get bored too.
 
that's a fun law. I think it's more difficult to get a duck to sit on your head long to cross the border, however.
 
4:32 PM
TX lets you in, but border patrol is authorized to shoot the duck
 
LOL.
 
But.... doesn't that mean they are hunting on duty?
 
yeah but it's filed as catching illegal immigrants
 
wat
 
#TalkingThroughMyMug
 
4:40 PM
> @Vogel612 in a little while, excuse me the delay, Christmas is disrupting the routine a bit.
 
well, @Mat'sMug you can't say they are racists or even a specisist. They discriminate equally.
 
@Duga pffft, excuses...
 
lol
 
@Mat'sMug xD
solution is quite easy
actually there are two options, which one do you prefer:
we change the installer .iss to copy the file at the package downloaded by NuGet (in which case, if in the future Nuget downloads a new one, the .iss won't compile saying it didn't find it and we can guess what was going on
or, we just manually copy the files to the \libs folder
 
I vote for nuget.
we shouldn't manually copy if we don't have to.
but you should make sure you use a relative path
so that build works anywhere
 
4:47 PM
so in this case I'll hardcode the nuget package folder in the .iss. If nuget changes something, InnoScript will complain
 
checking --- the folder path won't change? (e.g. it odesn't contain version in the folder)
 
@NelsonVides keep in mind that AppVeyor is running that script too :)
 
@this it does: in my case is \Rubberduck\packages\LibGit2Sharp.NativeBinaries.1.0.185\runtimes\win7-x64\nati‌​ve
Funny thing is that the version didn't change but nuget still changed the .dll inside
 
@SonGokussj4 You are already ahead of where I was when I started.
 
Hmmm. I was hoping it'd be just something like packages\LitGit2Sharp\runtime\win7-x64\native
I don't like the fact that there's a version, even though it didn't change when you updated
 
4:57 PM
me not touching anything on Nuget, that was the path that it gave me
 
did you ask for a specific versin?
or just Install-package LibGit@Sharp
 
what annoyes me is that Nuget changed the .dll, but it didn't change the path that contains it
@this that one
 
It should not
because see, we gonna to depend on it being same in all build environment
if path can change, hell will break loose
 
why the same version contains a different .dll? Feels weird.
 
cos they done wrong
 
4:59 PM
@this only the .iss would break saying that it didn't find that folder
in which case we would get reminded that the folder is now a different one
 
hm, don't we have a reference to the library in the RD?
 
don't know what other way to automatise the construction
@this hey, that makes sense...
actually all other .dlls are being built into the \bin\release and then taken from there into the InnoScript compile
 
bingo. so the libgit2sharp dll should go into that folder, too
check the Compile Action
then you can take that
it shouldn't be ... I don't know, not somewhere else.
 
@this sounds like it
I'll check that later, gotta go now
 
@Mat'sMug do you remember if there were any reason why we didn't simply include it int he release folder originally?
 
5:06 PM
I'm going to be a bit of for the next week, tomorrow my mother is visiting for Christmas and she'll meet my girlfriend's family and they'll need translators for this because they're natives to different languages... :D
 
@this not really. the only work I did on the SC feature was UI work
some justification for it probably dates back to v1.0, with Chris learning about C# and .NET
 
@NelsonVides At least you can filter out anything you don't want going through.
I wish I had that.
 
5:24 PM
Duck check: Because I have contributor rights, if I delete a remote/upstream/FooBar branch from VS that'd remove it from the RD Repo?
 
@IvenBach Yes.
 
This newfound responsibility scares me...
 
so long as you can reverse your mistake that's actually not such a huge issue...
 
git is distributed - it would be rather hard to push upstream and really break anything.
 
@Vogel612 Pretty hard to undelete a branch.
 
5:28 PM
and what about the history? We can't overwrite that, right?
so we can't do push upstream --force (or was it --rebase)
 
Nope.
Well, not on Next.
 
IOW, only @Mat'sMug (and maybe Chris?) can totally rewrite history).
But even if they did go bezerk and do that, we can just fork from our last good copy - they can't even touch our history.
(please do not go bezerk, ok, Mat?)
 
lol
 
5:44 PM
Mug. I'm looking to address your comment on PR3622. Would the following be along the lines of what you had in mind?
var inspectionSettings = new CodeInspectionSettings
{
    CodeInspections = new HashSet<CodeInspectionSetting>(new[]
    {
        new CodeInspectionSetting("DoNotShowInspection", "Do not show me", CodeInspectionType.LanguageOpportunities, CodeInspectionSeverity.DoNotShow, CodeInspectionSeverity.DoNotShow),
        new CodeInspectionSetting("HintInspection", "I'm a hint", CodeInspectionType.LanguageOpportunities, CodeInspectionSeverity.Hint, CodeInspectionSeverity.Hint),
        new CodeInspectionSetting("SuggestionInspection", "I'm a suggestion", CodeInspectionType.MaintainabilityAndReadab
 
yup
 
:+1: Ok.
 
the comment could be more like "// explicit sort required for InspectionSettingsViewModel" though
or something like it
IOW don't make the comment repeat what the code is already saying
(OrderBy/ThenBy)
 
[rubberduck-vba/Rubberduck] build for commit 36925d7d on unknown branch: AppVeyor build succeeded
 
How would I go about making the grid filterable? There's space after Inspection Severities to add an InputBox that could be used to filter on the Description. I'm not sure how that would attach to the InspectionSettings property though.
 

« first day (1292 days earlier)      last day (1888 days later) »