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

Update wp-calypso dependency and regenerate shrinkwrap #1581

Merged
merged 1 commit into from Feb 21, 2019

Conversation

dechov
Copy link
Contributor

@dechov dechov commented Feb 20, 2019

Due to the introduction of file: dependencies to wp-calypso in Automattic/wp-calypso@b4baa44, the dependency tracks a dedicated branch (feature/woocommerce-services-dependency) off of that commit's parent, rather than a snapshot of the master branch. For now, we will cherry-pick desired commits from master to this branch.

Note: I considered specifying the branch itself rather than a particular commit, as I figured we will essentially have full control over that branch, but in fact we will still benefit from a review process for introducing Calypso changes to the plugin build (and from stable builds in general, as @DanReyLop pointed out).

These commits have been cherry-picked to that branch, and included in this update:

Also, Automattic/wp-calypso@a0b2026 (prior to the branching) fixes a color scheme bug in which the "Buy & Print" button was var( --color-accent ) but its busy state was still $blue-medium.

Due to the introduction of 'file:' dependencies to wp-calypso master, this change pins the dependency to a snapshot of a dedicated branch.
Copy link
Contributor

@marcinbot marcinbot 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 :shipit:

Did some smoke testing on shipping and label settings.

I think the new Calypso branch with cherry-picked commits is a decent workaround.

@dechov
Copy link
Contributor Author

dechov commented Feb 21, 2019

I think the new Calypso branch with cherry-picked commits is a decent workaround.

Agreed – should have mentioned that the Calypso branch plan was proposed by @DanReyLop in p9oQ9f-7S-p2 #comment-198.

@dechov dechov merged commit 2fae4eb into master Feb 21, 2019
@dechov dechov deleted the update/wp-calypso-dependency-to-branch branch February 21, 2019 15:51
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.

None yet

2 participants