Skip to content
This repository has been archived by the owner on Feb 24, 2024. It is now read-only.

Fix test --force-migrations flag bug #1812

Merged
merged 3 commits into from Oct 30, 2019

Conversation

ypjama
Copy link
Contributor

@ypjama ypjama commented Oct 22, 2019

  • This fixes a bug that occurs when --force-migrations flag is used while also specifying the packages to be tested. e.g. buffalo test ./actions/ --force-migrations

* This fixes a bug that occurs when --force-migrations flag is used
while also specifying the packages to be tested. e.g. `buffalo test
./actions/ --force-migrations`
@ypjama ypjama requested a review from a team as a code owner October 22, 2019 19:22
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Found some fixes!

P.S. share your ideas, feedbacks or issues with us at https://github.com/fixmie/feedback (this message will be removed after the beta stage).

buffalo/cmd/test.go Outdated Show resolved Hide resolved
Suggested by fixmie

Co-Authored-By: fixmie[bot] <44270338+fixmie[bot]@users.noreply.github.com>
Copy link
Member

@mclark4386 mclark4386 left a comment

Choose a reason for hiding this comment

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

Code looks good, but having tested locally.

@markbates markbates added this to the v0.15.0 milestone Oct 30, 2019
@markbates markbates merged commit 4ad4272 into gobuffalo:development Oct 30, 2019
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.

None yet

3 participants