Skip to content

Remove assertion about allocation state as this varies #22

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 12, 2021
Merged

Remove assertion about allocation state as this varies #22

merged 1 commit into from
Feb 12, 2021

Conversation

biglovisa
Copy link
Contributor

What

Fix failing assertion in the Orders integration spec.

@biglovisa biglovisa self-assigned this Feb 10, 2021
@biglovisa biglovisa added the Needs Review The PR is ready to be reviewed label Feb 10, 2021
@biglovisa biglovisa requested a review from thdaraujo February 10, 2021 22:14
thdaraujo
thdaraujo approved these changes Feb 12, 2021
View reviewed changes
Copy link
Contributor

@thdaraujo thdaraujo left a comment

Choose a reason for hiding this comment

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

👍

@thdaraujo thdaraujo added Ready This PR is ready to ship! and removed Needs Review The PR is ready to be reviewed labels Feb 12, 2021
@biglovisa biglovisa merged commit b5178e2 into main Feb 12, 2021
@biglovisa biglovisa deleted the lovisa/fix-failing-orders-test branch February 12, 2021 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thdaraujo thdaraujo thdaraujo approved these changes

Assignees

@biglovisa biglovisa

Labels
Ready This PR is ready to ship!
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@biglovisa @thdaraujo