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

fix(create-gatsby): Improve install (#28318) #28406

Merged
merged 1 commit into from
Dec 1, 2020
Merged

Conversation

gatsbybot
Copy link
Collaborator

Backporting #28318 to the 2.28 release branch

(cherry picked from commit 4fbded2)

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Dec 1, 2020
import { stripIndent } from "common-tags"
import { trackCli } from "./tracking"
import crypto from "crypto"
import { reporter } from "./reporter"
Copy link
Contributor

Choose a reason for hiding this comment

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

There was merge conflict here due to neighbouring

import { setSiteMetadata } from "./site-metadata"

added by #28324 which is not backported and was merged before this PR

stripIndent`
${w(`🎉 `)}Your new Gatsby site ${c.bold(
data.project
)} has been successfully bootstrapped
)} has been successfully created
at ${c.bold(path.resolve(data.project))}.
Copy link
Contributor

Choose a reason for hiding this comment

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

There was merge conflict here due to change from path.resolve(data.project) to fullPath added by #28324 which is not backported and was merged before this PR


if (plugins.length) {
console.log(c.bold(`${w(`🔌 `)}Installing plugins...`))
reporter.info(`${w(`🔌 `)}Setting-up plugins...`)
await installPlugins(plugins, pluginConfig, path.resolve(data.project), [])
Copy link
Contributor

Choose a reason for hiding this comment

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

There was merge conflict here due to change from path.resolve(data.project) to fullPath added by #28324 which is not backported and was merged before this PR

@pieh pieh removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Dec 1, 2020
Copy link
Contributor

@ascorbic ascorbic left a comment

Choose a reason for hiding this comment

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

🚢

@pieh pieh merged commit 500ade2 into release/2.28 Dec 1, 2020
@delete-merged-branch delete-merged-branch bot deleted the backport-2.28-28318 branch December 1, 2020 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants