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

[Spree 2.1] Update spree revision to a version that doesnt depend on deface #5194

Merged

Conversation

luisramos0
Copy link
Contributor

What? Why?

After updating spree 2-1-stable in openfoodfoundation/spree#44 we need to update the spree revision in the 3-0-stable branch. The nokogiri upgrade will flow when we merge master into 3-0-stable

What should we test?

green build.

…allows us to upgrade nokogiri (done in another PR in the master branch)
@luisramos0 luisramos0 changed the title Update spre revision to a version that doesnt depend on deface, this … [Spree 2.1] Update spree revision to a version that doesnt depend on deface Apr 9, 2020
@luisramos0 luisramos0 self-assigned this Apr 9, 2020
@luisramos0 luisramos0 mentioned this pull request Apr 9, 2020
@@ -22,7 +22,7 @@ GIT

GIT
remote: https://github.com/openfoodfoundation/spree.git
revision: b7ad5b473f9e38c5a1882550b2b4e348f4824f1f
revision: f9264ebca0d8dd932a7dbee471514a3b85774c8f
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't this be openfoodfoundation/spree@73d0113 instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I used the merge commit: https://github.com/openfoodfoundation/spree/commits/2-1-0-stable/
I think it's the correct one right?

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah, right 👍

@sauloperez sauloperez merged commit 7b656fa into openfoodfoundation:3-0-stable Apr 16, 2020
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