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

Fix expected test data #172

Merged

rodneykinney merged 1 commit into main from fix-tests

May 24, 2023

Member

rodneykinney commented May 24, 2023

@soldni I see you fixed the deduper to write the id field. This updates the test data so that make test passes.

I didn't add make test to the automated tests, because it interacts with S3, but I run it manually before every merge.


          Fix expected tests data

4c51ca4

rodneykinney merged commit 434cf67 into main

10 checks passed

rodneykinney deleted the fix-tests branch

May 24, 2023 17:10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment