Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Tomek Wiszniewski
    @tomekwi
    Hi @stoeffel, do you have 2-3 minutes to help me out?
    Tomek Wiszniewski
    @tomekwi
    Well, when you do find a while – I have a problem with doxie tests.
    Everything passes smoothly on my machine
    but Travis says they’re broken: https://travis-ci.org/studio-b12/doxie/builds/67374100
    Could you please git checkout git@github.com:studio-b12/doxie; cd doxie; npm i; npm t?
    I’ve no idea what’s going on
    Tomek Wiszniewski
    @tomekwi
    Of course I meant git clone
    Christoph Hermann
    @stoeffel
    sure. I will check it out
    Tomek Wiszniewski
    @tomekwi
    Thanks. In the meantime we looked with @jimmyjjames on his machine – everything passes there as well :/
    Christoph Hermann
    @stoeffel
    why do you need process.timeout(500) in the test? could it be a timing issue?
    Tomek Wiszniewski
    @tomekwi
    I don’t think so: .timeout() fails with its own assertion and message. I want to be sure the process exits. If we don’t run process.exit(0) that should be the thing that fails.
    Christoph Hermann
    @stoeffel
    btw there is a bug in tape-spawn actual and expected are flipped
    Tomek Wiszniewski
    @tomekwi
    Thanks! I’ve spotted that as well :) maxogden/tape-spawn#5
    And we use the fork until the PR is merged
    Anyway @stoeffel, I think this might be a deeper problem with Travis
    Christoph Hermann
    @stoeffel
    :smile: you where just a little faster
    "I think this might be a deeper problem with Travis" yop could be.
    it works on my machine too.
    Tomek Wiszniewski
    @tomekwi
    when the tests pass on your machine, pass on mine and pass aith @jimmyjjames, then let’s ignore it
    Christoph Hermann
    @stoeffel
    :+1:
    Tomek Wiszniewski
    @tomekwi
    I’ll file a bug on travis when I find some time
    and in the meantime I’ll try to exclude the tests
    Christoph Hermann
    @stoeffel
    Tomek Wiszniewski
    @tomekwi
    BTW, I just need a couple of minutes and I’ll push the first functional version of doxie :D
    Thanks! Then we’re not the only ones that have such problems :)
    Christoph Hermann
    @stoeffel
    Cool ! :clap:
    btw did you try https://github.com/stoeffel/react-mini? any input would be cool?
    Tomek Wiszniewski
    @tomekwi
    The idea looks great! I’ve starred it as you may have seen
    but as a matter of fact I don’t use react
    Christoph Hermann
    @stoeffel
    oups didn't see that :flushed:
    Tomek Wiszniewski
    @tomekwi
    (wow, a poem :) )
    Christoph Hermann
    @stoeffel
    :smile:
    Tomek Wiszniewski
    @tomekwi
    Restarred :)
    studio-b12/doxie@ad5575e