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

Move gatsby site out of bolt repo #580

Merged
merged 1 commit into from
Dec 18, 2018
Merged

Move gatsby site out of bolt repo #580

merged 1 commit into from
Dec 18, 2018

Conversation

jesstelford
Copy link
Contributor

@jesstelford jesstelford commented Dec 18, 2018

Avoids a conflict with graphql packages, but still allows doing a
bolt install & running bolt website:start

Why

The graphql v14 update broke the gatsby website 😞

Gatsby depends on v0.13.x
So there ends up being multiple versions of graphql in node_modules (due to Bolt), which then gives this error:

error Cannot use GraphQLScalarType "JSON" from another module or realm.

Ensure that there is only one instance of "graphql" in the node_modules
directory. If different versions of "graphql" are the dependencies of other
relied on modules, use "resolutions" to ensure only one version is installed.

https://yarnpkg.com/en/docs/selective-version-resolutions

Duplicate "graphql" modules cannot be used at the same time since different
versions may have different capabilities and behavior. The data from one
version used in the function from another could produce confusing and
spurious results.

Use yarn's resolutions field doesn't work (bolt doesn't support it)

Avoids a conflict with `graphql` packages, but still allows doing a
`bolt install` & running `bolt website:start`
Copy link
Contributor

@timleslie timleslie left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@timleslie timleslie left a comment

Choose a reason for hiding this comment

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

👍

@jesstelford jesstelford merged commit a52a01d into master Dec 18, 2018
@jesstelford jesstelford deleted the fix-docs-site branch December 18, 2018 22:36
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.

2 participants