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

Cody: Log analytical events in the output logs #53870

Merged
merged 2 commits into from
Jun 21, 2023

Conversation

philipp-spiess
Copy link
Contributor

This PR makes sure all analytical event logs also appear in the output view. See my internal petition for some context.

Test plan

Screenshot 2023-06-21 at 16 48 19

@cla-bot cla-bot bot added the cla-signed label Jun 21, 2023
@github-actions github-actions bot added the team/code-exploration Issues owned by the Code Exploration team label Jun 21, 2023
@sourcegraph-bot
Copy link
Contributor

📖 Storybook live preview

@philipp-spiess philipp-spiess merged commit 5b0d56a into main Jun 21, 2023
22 checks passed
@philipp-spiess philipp-spiess deleted the ps/cody-log-all-analytics-events-in-verbose-log branch June 21, 2023 15:13
ErikaRS pushed a commit that referenced this pull request Jun 22, 2023
This PR makes sure all analytical event logs also appear in the output
view. See my [internal
petition](https://sourcegraph.slack.com/archives/C05AGQYD528/p1687354919604099)
for some context.

## Test plan

<img width="661" alt="Screenshot 2023-06-21 at 16 48 19"
src="https://github.com/sourcegraph/sourcegraph/assets/458591/4ae4a7a2-adb7-4102-a29d-0d56a7c35c94">


<!-- All pull requests REQUIRE a test plan:
https://docs.sourcegraph.com/dev/background-information/testing_principles
-->
sanderginn pushed a commit that referenced this pull request Jun 23, 2023
This PR makes sure all analytical event logs also appear in the output
view. See my [internal
petition](https://sourcegraph.slack.com/archives/C05AGQYD528/p1687354919604099)
for some context.

## Test plan

<img width="661" alt="Screenshot 2023-06-21 at 16 48 19"
src="https://github.com/sourcegraph/sourcegraph/assets/458591/4ae4a7a2-adb7-4102-a29d-0d56a7c35c94">


<!-- All pull requests REQUIRE a test plan:
https://docs.sourcegraph.com/dev/background-information/testing_principles
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed team/code-exploration Issues owned by the Code Exploration team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants