Skip to content

Conversation

squirly
Copy link
Contributor

@squirly squirly commented Apr 21, 2020

↪️ Pull Request

Fixes #4472
Fixes #4511

💻 Examples

See integration test

🚨 Test instructions

Includes integration tests

@height
Copy link

height bot commented Apr 21, 2020

Link Height tasks by mentioning a task ID in the pull request title or description, commit messages, or comments.

You can also use "Close T-X" to automatically close a task when the pull request is merged.

"format": "prettier --write \"./packages/*/*/{src,bin,test}/**/*.{js,json,md}\"",
"link-all": "node scripts/link-all.js packages",
"unlink-all": "node scripts/unlink-all.js packages",
"check": "flow check",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Would usually expect checks to have an associated run script for running locally. Can remove if this is unwanted.

Copy link
Contributor

Choose a reason for hiding this comment

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

I believe I use yarn flow check for this. Is there a particular need for a top-level script for this?

"format": "prettier --write \"./packages/*/*/{src,bin,test}/**/*.{js,json,md}\"",
"link-all": "node scripts/link-all.js packages",
"unlink-all": "node scripts/unlink-all.js packages",
"check": "flow check",
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe I use yarn flow check for this. Is there a particular need for a top-level script for this?

@DeMoorJasper DeMoorJasper merged commit 6d747bc into parcel-bundler:v2 Jun 13, 2020
@squirly squirly deleted the v2-graphql-named-imports branch June 13, 2020 18:45
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.

GraphQL : Export string instead of AST Support named imports in graphql transformer
4 participants