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 to the latest dev-support template and allow usage of Solidus 3.0 #46

Merged
merged 4 commits into from
Feb 17, 2021

Conversation

MinasMazar
Copy link
Contributor

@MinasMazar MinasMazar commented Dec 11, 2020

Update to solidus_dev_support 2.3.0. Also prepare extension for Solidus 3.0.

Refs.

@MinasMazar MinasMazar self-assigned this Dec 12, 2020
@MinasMazar MinasMazar requested a review from elia December 12, 2020 10:47
@MinasMazar MinasMazar force-pushed the mm/update_dev_support branch 2 times, most recently from 010e83b to ce62436 Compare January 22, 2021 00:49
@MinasMazar MinasMazar marked this pull request as ready for review January 22, 2021 08:59
Flavio Auciello added 3 commits January 22, 2021 18:21
- disable Rubocop rule `MultipleExpectations`
- correct Rubocop offence: avoid use of `let!`
- disable Rubocop rule: `RSpec/MultipleMemoizedHelpers`
- raise limit for `RSpec/NestedGroups` rule
- explicitly name example subject
- start context description with "when"
- prefer `have_received` to test spies
- appease [`Security/Open` rule][rubydoc-rule]

[rubydoc-rule]: https://www.rubydoc.info/gems/rubocop/RuboCop/Cop/Security/Open
This is using the new SolidusSupport helper method to determine when
using one or the other version.

Refs.
- [Solidus RFC](solidusio/solidus#3234)
- [Solidus 3.0 Extensions Hackathon](solidusio/solidus#3911)
@MinasMazar MinasMazar changed the title Update to the latest dev-support template Update to the latest dev-support template and allow usage of Solidus 3.0 Feb 5, 2021
Copy link
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

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

Thanks Flavio!

@kennyadsl kennyadsl merged commit 4d8dfbc into master Feb 17, 2021
@kennyadsl kennyadsl deleted the mm/update_dev_support branch February 17, 2021 13:51
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