Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Cody aggregated ping additions.#50835

Merged
nathan-downs merged 50 commits into
mainfrom
nd/add-cody-pings
May 3, 2023
Merged

Cody aggregated ping additions.#50835
nathan-downs merged 50 commits into
mainfrom
nd/add-cody-pings

Conversation

@nathan-downs

@nathan-downs nathan-downs commented Apr 18, 2023

Copy link
Copy Markdown
Contributor

Test plan

Verify by running locally and checking ping structure.
Verify that payload is properly formed and submitted to pubsub topic.

@cla-bot cla-bot Bot added the cla-signed label Apr 18, 2023
@nathan-downs nathan-downs marked this pull request as ready for review April 18, 2023 21:11
Comment thread cmd/frontend/internal/app/updatecheck/client.go Outdated
@unknwon

unknwon commented May 3, 2023

Copy link
Copy Markdown
Contributor

Thank you for the followups! Let's wait for CI.

@nathan-downs nathan-downs disabled auto-merge May 3, 2023 13:46
@unknwon

unknwon commented May 3, 2023

Copy link
Copy Markdown
Contributor

Comment thread cmd/frontend/internal/app/updatecheck/client.go Outdated
Comment thread cmd/frontend/internal/app/updatecheck/handler.go Outdated

@akalia25 akalia25 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! 🚀

@nathan-downs nathan-downs merged commit c5f62c0 into main May 3, 2023
@nathan-downs nathan-downs deleted the nd/add-cody-pings branch May 3, 2023 17:44
github-actions Bot pushed a commit that referenced this pull request May 3, 2023
## Test plan

Verify by running locally and checking ping structure.
Verify that payload is properly formed and submitted to pubsub topic.

<!-- All pull requests REQUIRE a test plan:
https://docs.sourcegraph.com/dev/background-information/testing_principles
-->

---------

Co-authored-by: Joe Chen <joe@sourcegraph.com>
(cherry picked from commit c5f62c0)
unknwon pushed a commit that referenced this pull request May 3, 2023
## Test plan

Verify by running locally and checking ping structure.
Verify that payload is properly formed and submitted to pubsub topic.

&lt;!-- All pull requests REQUIRE a test plan:
https://docs.sourcegraph.com/dev/background-information/testing_principles
--&gt;
 <br> Backport c5f62c0 from #50835

Co-authored-by: Nathan Downs <85511556+nathan-downs@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants