Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
    Artem Sapegin
    @sapegin
    Also weird, I don’t hat /Users/tema/webpack-blocks/packages/webpack/tsconfig.json.
    Andy Wermke
    @andywer
    That looks pretty weird 😳
    But works on my machine... 😅
    (Also didn't fail on Travis CI today)
    Artem Sapegin
    @sapegin
    Could you try to make a fresh clone?
    Andy Wermke
    @andywer
    Just did. Fresh clone + npm install + npm test. Everything ok
    Artem Sapegin
    @sapegin
    :cry:
    Trying to restart and clean npm cache (though it doesn’t like the idea).
    Andy Wermke
    @andywer
    webpack-blocks/packages/webpack on  master [!] via ⬢ v8.0.0
➜ npx ava

[at-loader] Using typescript@2.5.3 from typescript and "tsconfig.json" from /private/tmp/webpack-blocks/packages/webpack/t
sconfig.json.


[at-loader] Checking started in a separate process...

[at-loader] Ok, 0.001 sec.

  21 passed
    (when running npm test in the root directory I didn't see the typescript output)
    Artem Sapegin
    @sapegin
    Still cleaning the cache…
    Artem Sapegin
    @sapegin

    added 666 packages in 197.36s

    It was 15 s with cache :scream:

    Nope, same error ;-(
    Artem Sapegin
    @sapegin
    WTF?!
    ~
❯ npm ls @types/node
/Users/tema
└─┬ enzyme@3.1.0
  └─┬ cheerio@1.0.0-rc.2
    └─┬ parse5@3.0.2
      └── @types/node@6.0.88
    Andy Wermke
    @andywer
    is that the source of your troubles?
    Artem Sapegin
    @sapegin
    Yep, I had node_modules in my home folder since October 4 ;-|
    Andy Wermke
    @andywer
    Artem Sapegin
    @sapegin
    andywer/webpack-blocks#230
    And I’m going to bed — it’s too late already :sleeping:
    Artem Sapegin
    @sapegin

    @jvanbruegge Similar issue was reported at Styleguidist: https://gist.github.com/willhlaw/6134856f30a2bf076b829ebec8ee4fd9

    So something in Uglify plugin or webpack was changes but I don’t understand what.

    Artem Sapegin
    @sapegin
    @jvanbruegge Will you have time to finish this PR soon? andywer/webpack-blocks#231
    Jan van Brügge
    @jvanbruegge
    done
    Artem Sapegin
    @sapegin
    @jvanbruegge Could you please lock the version (remove ^) of uglifyjs-webpack-plugin?
    Jan van Brügge
    @jvanbruegge
    ok
    Artem Sapegin
    @sapegin
    Otherwise they will break it again in the next beta ;-0
    Artem Sapegin
    @sapegin
    http://wow.sapegin.me/073T1F2C3I0n/Image%202017-10-20%20at%203.47.23%20PM.png
    Andy Wermke
    @andywer
    Ohhh 🎉
    At work?
    Artem Sapegin
    @sapegin
    My last commit before two weeks holiday ;-)
    Yep!
    Vlad Zhukov
    @vlad-zhukov
    About the uglify block:
    it's not nice to have a broken block for so long time, what can I help you with so you can release a new version?
    Jan van Brügge
    @jvanbruegge
    in theory it's working (i updated my boilerplate to force the options), but somehow the tests fail
    and I hadnt had the time to investigate
    Andy Wermke
    @andywer
    you are right, @vlad-zhukov. but fixing the build turns out to be quite tricky.
    i just pushed a change to your feature branch, @jvanbruegge, using uglifyjs-webpack-plugin v1.0.1. if the build still fails on node 6, i will just deactivate the failing test for now...
    Jan van Brügge
    @jvanbruegge
    ok
    Andy Wermke
    @andywer
    Good news: Could fix the build in a second PR that includes the uglifyjs-webpack-plugin options fix, disables one test and moves/renames the test-app to packages/sample-app, so lerna can link the local packages
    https://travis-ci.org/andywer/webpack-blocks/builds/292615756
    Andy Wermke
    @andywer
    Can we live with releasing webpack-blocks v1.1.0, shipping the updated uglifyjs-webpack-plugin with the nasty breaking change?
    Actually it would probably rather be webpack-blocks v1.0.0-rc.2 which is even worse
    Good news: Just checked the dependents (fortunately it's only two of them right now and they cause pretty much all downloads right now) and they shouldn't be affected by the change
    So I guess webpack-blocks v1.0.0-rc.2 it is for shipping the uglify fix
    Vlad Zhukov
    @vlad-zhukov
    Hey, would be nice to have some feedback on andywer/webpack-blocks#236
    Andy Wermke
    @andywer
    Hey Vlad. Thx for the PR! :)
    Good ideas. I'd just like to move updating the dependenicies and refactoring the ugly unit test into separate PRs
    Vlad Zhukov
    @vlad-zhukov
    I've found a reason why CI is failing. Travis runs lerna-bootstrap, it upgrades deps across all packages (e.g. typescript), and it fails to build
    This is basically why yarn exists :smirk:
    It's not possible to fix it right now without publishing a new version of typescript block
    Andy Wermke
    @andywer
    Is it a new TypeScript version that crashes the build?
    Vlad Zhukov
    @vlad-zhukov
    kind of
    I wasn't correct about publishing a new version, it's fixable
    The new typescript requires @types/node that are missing in the packages/webpack block
    I've fixed the typescript bug, now fighting an elm one