Skip to content
This repository has been archived by the owner on Jun 21, 2024. It is now read-only.

fix capture CI #32

Merged
merged 4 commits into from
May 2, 2024
Merged

fix capture CI #32

merged 4 commits into from
May 2, 2024

Conversation

xvello
Copy link
Contributor

@xvello xvello commented May 2, 2024

  • Use our own updated fork of axum-test-helper, as the one we used has been deleted. I tried moving django_compat.rs to the capture-server test harness first, but got into complications
  • Fix the sent_at comparaison, the CI on capture: don't serialize sent_at if empty #28 should have failed 🤔

@xvello xvello requested a review from a team May 2, 2024 09:59
@xvello xvello enabled auto-merge (squash) May 2, 2024 10:04
@xvello xvello merged commit db332e5 into main May 2, 2024
4 checks passed
@xvello xvello deleted the xvello/axumup branch May 2, 2024 10:04
@tomasfarias tomasfarias mentioned this pull request May 2, 2024
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tomasfarias tomasfarias tomasfarias approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xvello @tomasfarias