Closed mhauru closed 6 months ago
I tagged y'all as reviewers, but maybe hold off until #207 is merged, because right now the diff is entirely unworkable.
With the FastAPI PR merged, I reset the base branch, so the diff should now make sense.
@mhauru - Could you quickly talk through what you expect from the user? Service Name is explanatory, but URL? Does this mean the service needs to call an API?
Yes, the URL is the URL to which a REST request should be sent when the service is to be run. E.g. the URL for triggering an Azure function to run.
Attention: Patch coverage is 86.90141%
with 93 lines
in your changes are missing coverage. Please review.
Project coverage is 89.31%. Comparing base (
ae28d48
) to head (952e25a
).:exclamation: Current head 952e25a differs from pull request most recent head 8c17f43. Consider uploading reports for the commit 8c17f43 to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Adds
No scheduling yet.
Builds on #207, so getting that merged first will make the diff manageable.