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

[v3.2] Fix rake error testing the update generator #4981

Merged
merged 2 commits into from
Mar 16, 2023

Conversation

github-actions[bot]
Copy link

Backport

This will backport the following commits from master to v3.2:

Questions ?

Please refer to the Backport tool documentation

It looks like the `spree:install:migrations` task is not available when
running through `Rails::Generators.invoke`, as it context differs from
common shell invocation.

We fix that by not trying to run the task on test environment. In the
end, it wouldn't be a good idea to copy the migrations at the test suite
runtime.

Fixes #4978

(cherry picked from commit 3bfcac6)
(cherry picked from commit fb01309)
@github-actions github-actions bot requested a review from a team as a code owner March 16, 2023 04:26
@waiting-for-dev waiting-for-dev changed the title [v3.2] Merge pull request #4980 from nebulab/waiting-for-dev/fix_missing_rake_task_warning [v3.2] Fix rake error testing the update generator Mar 16, 2023
@waiting-for-dev waiting-for-dev added the changelog:solidus_core Changes to the solidus_core gem label Mar 16, 2023
@waiting-for-dev waiting-for-dev merged commit 2ea8e9a into v3.2 Mar 16, 2023
@waiting-for-dev waiting-for-dev deleted the backport/v3.2/pr-4980 branch March 16, 2023 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:solidus_core Changes to the solidus_core gem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant