Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add --autoclose to automatically close the browser tab opened b… #3059

Closed
wants to merge 1 commit into from
Closed

Conversation

Taym95
Copy link
Contributor

@Taym95 Taym95 commented May 20, 2019

This is my first contribution to parcel I tied to fix #3027

↪️ Pull Request

💻 Examples

🚨 Test instructions

✔️ PR Todo

  • Added/updated unit tests for this change
  • Filled out test instructions (In case there aren't any unit tests)
  • Included links to related issues/PRs

@Taym95
Copy link
Contributor Author

Taym95 commented May 21, 2019

The node 6 tests failed for me before my commit! does anyone know why?

@Kogia-sima
Copy link
Contributor

@Taym95 Have you seen this issue? In web worker context the window object is not defined and then causes an error.

Also you should bind HMRServer.stop() method to SIGINT signal, otherwise Ctrl + C does not work.

Copy link
Member

@DeMoorJasper DeMoorJasper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, tests are unstable though (but this isn't because of this PR)

Parcel 1 is feature locked though so we can focus on Parcel 2 and so we can keep up with Parcel 1 changes...

@DeMoorJasper
Copy link
Member

feel free to re-open this for v2, gonna close this as v2 is feature locked

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants