Skip to content

Make launch editor a standalone package. #4091

Closed

Description

@cpojer

The launchEditor module is super useful: https://github.com/facebook/create-react-app/blob/next/packages/react-dev-utils/launchEditor.js

I'd like to use it standalone, outside of create-react-app. Is there anyone willing to send a PR to make this a standalone package? :)

Activity

  1. egoist commented on Feb 28, 2018

    @egoist
  2. cpojer commented on Feb 28, 2018

    @cpojer
    ContributorAuthor

    Oh cool! Nevermind then – wondering why create-react-app isn't using this though.

  3. SimenB commented on Feb 28, 2018

    @SimenB
    Contributor
  4. Timer commented on Feb 28, 2018

    @Timer
    Contributor

    It looks like we added that file mere days after sindresorhus created his module, May 4th vs 14th so we probably didn't know about it.
    launch-editor is extracted from our code base, so its effectively the same thing.

    We should probably turn this into a module, it looks like open-editor doesn't have as much editor support and this would just be an official version of launch-editor.

  5. reopened this on Feb 28, 2018
  6. added this to the milestone on Feb 28, 2018
  7. gaearon commented on Feb 28, 2018

    @gaearon
    Contributor

    wondering why create-react-app isn't using this though

    I don't think @yyx990803 reached out to suggest extracting it before forking. We'd be happy to collab on this and keep it as a separate package.

  8. gaearon commented on Feb 28, 2018

    @gaearon
    Contributor

    (Didn't mean it in a bad way—I also fork a lot of stuff all the time because it takes time to establish relationships etc. But I'd be happy to give @yyx990803 commit rights in this repo and continue maintaining it here :-)

  9. yyx990803 commented on Feb 28, 2018

    @yyx990803
    Contributor

    Yeah, I just assumed create-react-app kept it inside react-dev-utils for a reason (like all the other utils in there) so didn't make that suggestion.

    Another reason I forked it is mostly because I may need to make changes that make it incompatible with the needs of create-react-app. If you guys are happy with the current API of launch-editor I'd be happy to collaborate.

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions