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

test/purge: assert database state after purging #1178

Merged
merged 8 commits into from
Dec 4, 2024

Conversation

alxndrsn
Copy link
Contributor

@alxndrsn alxndrsn commented Aug 26, 2024

No description provided.

@alxndrsn

This comment was marked as outdated.

@alxndrsn

This comment was marked as resolved.

@alxndrsn alxndrsn force-pushed the purge-test-extensions branch from 18d969f to ec54868 Compare October 9, 2024 06:10
@matthew-white
Copy link
Member

@alxndrsn, I just wanted to check about the status of this PR. I see that the PR is approved. Is it ready to merge? Or are you waiting for feedback from @ktuite?

@alxndrsn alxndrsn marked this pull request as ready for review November 20, 2024 10:58
@alxndrsn
Copy link
Contributor Author

@matthew-white I think this is ready to merge, unless you have reservations.

@matthew-white
Copy link
Member

No reservations from my end! I haven't taken a look at the code, I was just checking about the status of the PR.

@alxndrsn alxndrsn merged commit 663e3b9 into getodk:master Dec 4, 2024
1 check passed
@alxndrsn alxndrsn deleted the purge-test-extensions branch December 4, 2024 09:11
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.

3 participants