8000 Make puppeteer peerDependency · Issue #108 · peterbe/minimalcss · GitHub
[go: up one dir, main page]

Skip to content

Make puppeteer peerDependency #108

Open

Description

@stereobooster

If I use minimalcss in react-snap it will download two copies of puppeteer. If we make minimalcss-core without CLI and with peerDependency puppeteer and minimalcss with CLI and with puppeteer+minimalcss-core dependency I will be able to use minimalcss-core in react-snap with one copy of puppeteer

Todo this we probably want to use monorepo, see https://github.com/lerna/lerna and https://yarnpkg.com/blog/2017/08/02/introducing-workspaces/

Activity

  1. peterbe commented on Jan 29, 2018

    @peterbe
    Owner

    I have only read about workspaces but if it means we can have one package.json for the cli and one package.json for the "core", that would be awesome.
    But what does it mean in terms of npmjs.com?
    I do like the idea of splitting the code into smaller parts but I'm quite pessimistic about breaking up the repo.

  2. stereobooster commented on Jan 29, 2018

    @stereobooster
    CollaboratorAuthor

    I have only read about workspaces but if it means we can have one package.json for the cli and one package.json for the "core", that would be awesome.

    Yes.

    But what does it mean in terms of npmjs.com?

    It means you will have two separate npm packages, but they will live in one repo for development convenience.

    I do like the idea of splitting the code into smaller parts but I'm quite pessimistic about breaking up the repo.

    Create-react-app does this, for example.

  3. peterbe commented on Jan 29, 2018

    @peterbe
    Owner

    Neat! I like this.
    Do I need to run jest inside each directory? (not a problem)

  4. stereobooster commented on Jan 29, 2018

    @stereobooster
    CollaboratorAuthor

    Do I need to run jest inside each directory?

    Yes, but there will be one top level package.json you can create one task which will run all tests in subdirectories

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      0