-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add CI workflow to build and test docker image #25
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I merged main
, which has a fix for the request parsing issues
@@ -15,7 +15,7 @@ | |||
"url": "url 2" | |||
} | |||
], | |||
"click_data": [ | |||
"click_histories": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like we've fixed this six times this week already, but...reverting the big dependency management change undid it again 😓
This reverts commit c5c8b42.
To reduce risk of future errors as we work on refactoring and other improvements, this PR does some minimal docker cleanups and adds docker-based testing of the repo to main-branch pushes and PRs.
Doesn't work yet due to outstanding problems with request formats.