Skip to content
This repository was archived by the owner on Sep 12, 2019. It is now read-only.

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Jan 11, 2017

Solidus 1.3 introduced persisted shipping rate taxes. This updates the shipping_rate spec to reflect the changes made in Solidus 1.3.

Also makes sure that this extension only installs for 1.x series of Solidus, as it is not compatible to Solidus 2.x yet.

Solidus 1.3 introduced persisted shipping rate taxes. This updates the shipping_rate spec to reflect the changes made in Solidus 1.3.

Also makes sure that this extension not installs for Solidus v2.1, as it is not compatible to Solidus 2.1 yet.
Rake 2.0 and Rails 5 need at least Ruby 2.2.2
Copy link

@mamhoff mamhoff left a comment

Choose a reason for hiding this comment

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

We both worked on the spec, so I'm good with that :) - I'm also fine with the newer Ruby version. Before merge I'd like to see this Gem working somewhere.

@mamhoff
Copy link

mamhoff commented Jan 11, 2017

Ok, so the test run indicates a flaky spec where the order of an array matters more than it should.

@jacobherrington
Copy link

@tvdeyen is this and #25 something we still want to get merged in? Is it worth getting this extension up to 2.7?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants