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

fix: remove test fixture fit_result #355

Merged
merged 1 commit into from
Nov 29, 2021

Conversation

redeboer
Copy link
Member

Fix-up to #349

Writing with a callback in a pytest fixture and then loading it back in a test led to instable CI. This PR should fix that.

Writing with a callback in a fixture and then loading it in a test led
to instable CI. This should fix that
@redeboer redeboer added the 🐛 Bug Something isn't working label Nov 29, 2021
@redeboer redeboer added this to the 0.4.0 milestone Nov 29, 2021
@redeboer redeboer self-assigned this Nov 29, 2021
@redeboer redeboer added the 🖱️ DX Improvements to the Developer Experience label Nov 29, 2021
@redeboer redeboer enabled auto-merge (squash) November 29, 2021 16:13
@redeboer redeboer merged commit a971f77 into ComPWA:main Nov 29, 2021
@redeboer redeboer deleted the improve-integration-tests branch November 29, 2021 16:23
@redeboer redeboer changed the title fix: remove test fixture fit_result fix: remove test fixture fit_result Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working 🖱️ DX Improvements to the Developer Experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant