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 unit tests #488

Merged
merged 1 commit into from
Dec 2, 2024
Merged

Fix unit tests #488

merged 1 commit into from
Dec 2, 2024

Conversation

shish
Copy link
Collaborator

@shish shish commented Dec 2, 2024

These bit-rotted, let's bring them up to date

These bit-rotted, let's bring them up to date
shish added a commit to shish/safe that referenced this pull request Dec 2, 2024
After thecodingmachine#486 thecodingmachine#487 and thecodingmachine#488 are merged, running `./.devcontainer/build.sh` produces this output, which contains a couple of years worth of updates - and from there, the github cronjob _should_ submit incremental updates as PRs automatically
@shish
Copy link
Collaborator Author

shish commented Dec 2, 2024

(Tests are failing in this PR because the tests now check for the actual correct data - but we aren't generating the correct data - #486 and #487 take care of those, and with all three combined, we should have green tests)

@staabm staabm merged commit 26b520c into thecodingmachine:master Dec 2, 2024
7 of 12 checks passed
@shish shish deleted the tests branch December 2, 2024 19:25
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.

2 participants