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.

Sign up for GitHub

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

Jump to bottom

cody: inline assist papercuts #53980

Merged
merged 3 commits into from
Jun 26, 2023
Merged

cody: inline assist papercuts #53980

merged 3 commits into from
Jun 26, 2023

Conversation

abeatrix
Copy link
Contributor

Close #53801 #53978 #53977 #53979

WIP

Test plan

All tests should be passing because these are minor visual changes that do not affect current tests

@cla-bot cla-bot bot added the cla-signed label Jun 22, 2023
@sourcegraph-bot
Copy link
Contributor

sourcegraph-bot commented Jun 22, 2023 •
edited

📖 Storybook live preview

philipp-spiess
philipp-spiess approved these changes Jun 22, 2023
View reviewed changes
Copy link
Contributor

@philipp-spiess philipp-spiess left a comment

Choose a reason for hiding this comment

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

Stamping this but there's some weird issue for me in the .js file, we're chatting via DM too

client/cody/package.json Outdated Show resolved Hide resolved
client/cody/src/services/DecorationProvider.ts Outdated Show resolved Hide resolved
@abeatrix abeatrix merged commit c076977 into main Jun 26, 2023
23 of 24 checks passed
@abeatrix abeatrix deleted the bee/cody-inline-papercuts branch June 26, 2023 16:47
This was referenced Jun 26, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@toolmantim toolmantim toolmantim left review comments

@philipp-spiess philipp-spiess philipp-spiess approved these changes

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cody Inline Assist: Do not change current focus when fixup is done
4 participants
@abeatrix @sourcegraph-bot @toolmantim @philipp-spiess