Skip to content
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

Shawn/update throttle feat branch #1234

Merged

Conversation

shaspitz
Copy link
Contributor

I need to update my feature branch to main

MSalopek and others added 2 commits August 24, 2023 14:30
* docs: add testing improvements ADR

* docs: fix links and typos

* Update docs/docs/adrs/adr-011-improving-test-confidence.md

Co-authored-by: Shawn <44221603+smarshall-spitzbart@users.noreply.github.com>

* Update docs/docs/adrs/adr-011-improving-test-confidence.md

Co-authored-by: Shawn <44221603+smarshall-spitzbart@users.noreply.github.com>

* docs: update from comments

---------

Co-authored-by: Shawn <44221603+smarshall-spitzbart@users.noreply.github.com>
* consumer proto folder

* provider comments, move one msg

* ccv comments

* moved shiz

* proto changes to build

* make proto gen again

* no circ dep

* Update wire.proto

* comments

* dun builds

* progress save

* FIN

* refactors

* rm improper proto ref

* rm todos, issue now

* lint till I die

* update comments, rebuild protos

* change ccv.go to wire.go, and test file

* consistent comment

* Update wire.proto

* example alias diff
@shaspitz shaspitz requested a review from a team as a code owner August 25, 2023 02:07
@github-actions github-actions bot added C:Testing Assigned automatically by the PR labeler C:x/consumer Assigned automatically by the PR labeler C:x/provider Assigned automatically by the PR labeler C:x/types Assigned automatically by the PR labeler C:Docs Assigned automatically by the PR labeler C:ADR Assigned automatically by the PR labeler labels Aug 25, 2023
@mpoke mpoke merged commit 8056391 into feat/throttle-with-retries-complete Aug 25, 2023
27 of 30 checks passed
@mpoke mpoke deleted the shawn/update-throttle-feat-branch branch August 25, 2023 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:ADR Assigned automatically by the PR labeler C:Docs Assigned automatically by the PR labeler C:Testing Assigned automatically by the PR labeler C:x/consumer Assigned automatically by the PR labeler C:x/provider Assigned automatically by the PR labeler C:x/types Assigned automatically by the PR labeler
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants