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

Setup basic API #2

Merged
merged 4 commits into from
Dec 7, 2023
Merged

Setup basic API #2

merged 4 commits into from
Dec 7, 2023

Conversation

ellie
Copy link
Contributor

@ellie ellie commented Dec 4, 2023

Needs routes to hook into pgqueue obv, but

  1. Pull out common dependencies, either currently common or likely future common
  2. Add http metrics
  3. Start up axum server
  4. Add config

@bretthoerner bretthoerner self-requested a review December 6, 2023 16:20
bretthoerner
bretthoerner approved these changes Dec 6, 2023
View reviewed changes
hook-producer/src/metrics.rs Show resolved Hide resolved
hook-producer/src/metrics.rs Show resolved Hide resolved
@bretthoerner
Copy link
Contributor

Hm, I didn't expect the Unresolved conversations checks. That seems kind of annoying.

@tomasfarias tomasfarias mentioned this pull request Dec 7, 2023
Merged
@ellie
Copy link
Contributor Author

ellie commented Dec 7, 2023

Hm, I didn't expect the Unresolved conversations checks. That seems kind of annoying.

Oof yeah, I'll disable it

@ellie ellie merged commit a0072d1 into main Dec 7, 2023
4 checks passed
@bretthoerner bretthoerner deleted the ellie/setup-producer-api branch January 17, 2024 15:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tomasfarias tomasfarias tomasfarias left review comments

@bretthoerner bretthoerner bretthoerner 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.
3 participants
@ellie @bretthoerner @tomasfarias