Skip to content

feat: bootstrap with initial webhook receiver logic#1

Merged
cbartz merged 63 commits intomainfrom
bootstrap
Sep 24, 2025
Merged

feat: bootstrap with initial webhook receiver logic#1
cbartz merged 63 commits intomainfrom
bootstrap

Conversation

@cbartz
Copy link
Copy Markdown
Collaborator

@cbartz cbartz commented Sep 16, 2025

PR to add

  • chore files like README, github workflows
  • logic for webhook-gateway application (without redelivery action)

Follow-up PRs to add opentelemetry and 12F charm for the webhook-gateway application are planned

Copy link
Copy Markdown
Collaborator Author

@cbartz cbartz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comments coming from review call

Comment thread webhook-gateway/main.go Outdated
Comment thread internal/webhook/server.go Outdated
Comment thread internal/webhook/server.go Outdated
Comment thread internal/webhook/server.go
Comment thread internal/webhook/server_test.go Outdated
Comment thread internal/webhook/server.go Outdated
Comment thread internal/queue/queue.go Outdated
@cbartz cbartz requested a review from Copilot September 23, 2025 10:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 15 out of 16 changed files in this pull request and generated 3 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread webhook-gateway/main.go Outdated
Comment thread internal/webhook/server.go Outdated
Comment thread internal/queue/producer.go Outdated
@cbartz cbartz requested a review from Copilot September 23, 2025 10:15
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 15 out of 16 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@cbartz cbartz marked this pull request as ready for review September 23, 2025 10:16
Copy link
Copy Markdown
Member

@yanksyoon yanksyoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

Comment thread internal/queue/producer.go Outdated
Comment thread internal/queue/types.go Outdated
Comment thread internal/webhook/server.go
Comment thread internal/queue/types.go Outdated
Comment thread internal/queue/types.go Outdated
Comment thread internal/webhook/server.go Outdated
Comment thread webhook-gateway/main.go Outdated
Comment thread webhook-gateway/main.go Outdated
@cbartz cbartz requested a review from weiiwang01 September 24, 2025 06:29
@cbartz cbartz merged commit 45c79e3 into main Sep 24, 2025
2 checks passed
@cbartz cbartz deleted the bootstrap branch September 24, 2025 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants