by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jun 01 20:36
    nicolashenry synchronize #325
  • Jun 01 20:36
    nicolashenry synchronize #325
  • Jun 01 19:15
    nicolashenry commented #325
  • Jun 01 19:07
    nicolashenry edited #325
  • May 24 01:24
    nicolashenry review_requested #325
  • May 22 17:07
    nicolashenry commented #325
  • May 22 16:55
    nicolashenry synchronize #325
  • May 22 16:36
    nicolashenry synchronize #325
  • May 22 16:30
    nicolashenry synchronize #325
  • May 21 03:23
    nicolashenry ready_for_review #325
  • May 21 03:09
    nicolashenry synchronize #325
  • May 21 02:51
    nicolashenry synchronize #325
  • May 20 06:41
    ljharb closed #324
  • May 20 06:41
    ljharb closed #323
  • May 20 06:41

    ljharb on master

    fix(git): rename test directori… (compare)

  • May 19 23:51
    nicolashenry opened #325
  • May 15 03:12
    nicolashenry opened #324
  • May 13 19:57
    ljharb commented #323
  • May 13 19:41
    nicolashenry commented #323
  • May 13 19:39
    nicolashenry commented #323
Andreas Windt
@ta2edchimp
lodash/lodash#2177
this is where I saw those issues with asynchronously running those scripts ...
Sarbbottam Bandyopadhyay
@sarbbottam
cool
Andreas Windt
@ta2edchimp
Maybe & wait is something to think about, but on the other hand, I see no need to do that asynchronously tbh.
Sarbbottam Bandyopadhyay
@sarbbottam
why haven't I used npm-run-all :worried:
Andreas Windt
@ta2edchimp
Kent C. Dodds
@kentcdodds
Big fan of npm-run-all
Sarbbottam Bandyopadhyay
@sarbbottam
yep, seen it, but does it allow combinations like npm-run-all?
yea npm-run-all is awesome
BTW, have you guys seen scripty - https://github.com/testdouble/scripty?
Andreas Windt
@ta2edchimp
nope, not yet... but seems helpful to not clutter the package.json with loads of script statements ...
so, I'm off for bbq dinner with loads of moscow mule now.
wish you a nice weekend, if we don't hear/read from one another o/
Sarbbottam Bandyopadhyay
@sarbbottam
that ginger drink is great, enjoy your dinner and have a great weekend
Kent C. Dodds
@kentcdodds
I’ve seen scripty. Haven’t tried it. I don’t think I’ve run into a situation to justify the abstraction though ¯\_(ツ)_/¯
Sarbbottam Bandyopadhyay
@sarbbottam
yea, its a indication of build fatigue grunt > gulp > npm scripts > scripty :panda_face:
Kent C. Dodds
@kentcdodds
Haha
Sarbbottam Bandyopadhyay
@sarbbottam
@ta2edchimp still awake?
Andreas Windt
@ta2edchimp
partly :trollface:
Sarbbottam Bandyopadhyay
@sarbbottam
lol; great work on the verbose diff, I realized it when using, that its not that helpful.
Andreas Windt
@ta2edchimp
I have to admit, I thought, I would have attached a screenshot to #47
Sarbbottam Bandyopadhyay
@sarbbottam
yea, that would be great
I use licecap to create gif, when it is behavior
Andreas Windt
@ta2edchimp
But I'm with you on the general difference topic. I stumbled upon it when doing #47 and thought, "well that's strange"... I totally forgot about when comparing to the available rules xD
Sarbbottam Bandyopadhyay
@sarbbottam
http://www.cockos.com/licecap/ you might find it handy
Andreas Windt
@ta2edchimp
Yeah, will do... I thought of putting those into the README.md
Sarbbottam Bandyopadhyay
@sarbbottam
:+1:
Andreas Windt
@ta2edchimp
How do we do the default eslint-diff-rules?
Group the diffin' rules into groups? Like this:
diff rules

in config1, but not in config2:
foo-rule, bar-rule

in config2, but not in config1:
baz-rule
Sarbbottam Bandyopadhyay
@sarbbottam
looks good
Sarbbottam Bandyopadhyay
@sarbbottam
Screen Shot 2016-04-17 at 11.19.09 PM.png
@ta2edchimp I am loving the current output
Thanks!
Andreas Windt
@ta2edchimp
😁
Andreas Windt
@ta2edchimp
@sarbbottam a couple of questions, jut answer them when you got time to think about it (I'll have some time tomorrow to continue working on the main PR):are you ok with #56 besides it requiring a rebase? If yes, I propose
Bzzz ... fast return is fast
(cont) ... If yes, I propose I merge it tomorrow after rebasing.
And then continuing work on the main PR.
And final question: Do you prefer the cliui util in a separate PR, too?
Sarbbottam Bandyopadhyay
@sarbbottam
Hey sorry, was not logged in, just seen the notification in mail.
If yes, I propose I merge it tomorrow after rebasing
Please go ahead

Do you prefer the cliui util in a separate PR, too?

I would love that

Andreas Windt
@ta2edchimp
:+1: will do. Just got no time tonight, but tomorrow is almost only waiting for a client and a bit of chatting with him so I guess I'll have plenty of time for this to get it forward.
Sarbbottam Bandyopadhyay
@sarbbottam
awesome
Andreas Windt
@ta2edchimp
what the ...
merging #56 triggered a new release — it's clear that it did this, but it also did include the changes of #55, which did not trigger a new release itself... why? any idea?
Sarbbottam Bandyopadhyay
@sarbbottam
@ta2edchimp are you interested in data structures?
Andreas Windt
@ta2edchimp
I'd say "depends" 😜
What are you up to? (Sorry for the delayed response, I'm still a bit too busy over the next few days) ...
Sarbbottam Bandyopadhyay
@sarbbottam
I am trying to create a repo for Data Structures in javascript, for learning and reference perspective
https://github.com/sarbbottam/js-ds, so far I have implemented Stack only :p
Andreas Windt
@ta2edchimp
Uh nice! I'll definitely
... keep an eye on it (is that even a real sentence in english?)