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

Update file structure for cody #49767

Merged
merged 5 commits into from
Mar 21, 2023
Merged

Update file structure for cody #49767

merged 5 commits into from
Mar 21, 2023

Conversation

abeatrix
Copy link
Contributor

@abeatrix abeatrix commented Mar 21, 2023

This PR removes all the unused and outdated directories from the client/cody project as they have been integrated into the main app, and moves the VS Code extension to the base client/cody directory.

I have also updated the README.md in client/cody to reflect the changes.

All CSS files are now using relative units instead of hard-coded units. Everything should scale according to user's system.

Some minor CSS fixes.

Before After
image image
Before After
image image

Replaced placeholder logo with logo for cody:
image

Test plan

Run the following commands from the root of this repository in this branch:

  • pnpm --filter cody run build
  • pnpm --filter cody run test:unit
  • pnpm --filter cody run test:integration
  • pnpm --filter cody run vsce:package

Everything should work as before the move.

App preview:

Check out the client app preview documentation to learn more.

@github-advanced-security
Copy link

You have successfully added a new SonarCloud configuration ``. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab.

@abeatrix abeatrix requested a review from novoselrok March 21, 2023 17:18
Copy link
Contributor

@novoselrok novoselrok left a comment

Choose a reason for hiding this comment

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

LGTM!

I tried running the unit tests and integration tests locally, both are green. Building and running the extension locally works as well.

@abeatrix abeatrix merged commit 62c40f8 into cody-main Mar 21, 2023
@abeatrix abeatrix deleted the bee/cody-restructure branch March 21, 2023 19:21
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.

2 participants