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

Fixed a bug where the post-install script would suggest to setup a cu… #1734

Merged
merged 1 commit into from Sep 21, 2019

Conversation

LukasKiederle
Copy link
Contributor

@LukasKiederle LukasKiederle commented Sep 21, 2019

…stom origin when the project was already setup like this (#1731)

Closes #1731

Description

I changed the relativ path to the project to an absolute path because the hash which is used for comparison would not lead to the expected result.

Running yarn install on an already installed instance will not lead to this console output anymore: Please run Set up a custom origin for your copy of venia-concept: yarn buildpack create-custom-origin packages/venia-concept. )

@magento-cicd2
Copy link

magento-cicd2 commented Sep 21, 2019

CLA assistant check
All committers have signed the CLA.

@m2-community-project m2-community-project bot added this to Ready for Review in Pull Request Progress Sep 21, 2019
@PWAStudioBot
Copy link
Contributor

PWAStudioBot commented Sep 21, 2019

Fails
🚫 Missing "Verification Steps" section. Please add it back, with detail.
🚫 A version label is required. A maintainer must add one.
Messages
📖

Access a deployed version of this PR here. Make sure to wait for the "pwa-pull-request-deploy" job to complete.

📖 DangerCI Failures related to missing labels/description/linked issues/etc will persist until the next push or next nightly build run (assuming they are fixed).

If your PR is missing information, check against the original template here. At a minimum you must have the section headers from the template and provide some information in each section.

Generated by 🚫 dangerJS against e3282b9

…stom origin when the project was already setup like this. (magento#1731)
@zetlen zetlen self-requested a review September 21, 2019 11:17
@zetlen zetlen added the version: Patch This changeset includes backwards compatible bug fixes. label Sep 21, 2019
Copy link
Contributor

@zetlen zetlen left a comment

Choose a reason for hiding this comment

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

Perfect! Thank you, @LukasKiederle !

@zetlen zetlen merged commit 9548444 into magento:develop Sep 21, 2019
@m2-community-project m2-community-project bot moved this from Ready for Review to Done in Pull Request Progress Sep 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
version: Patch This changeset includes backwards compatible bug fixes.
Development

Successfully merging this pull request may close these issues.

[bug]: Post-install script should stop suggesting custom origin setup once it's complete
4 participants