Skip to content

major refactor: on switching to a local cmd app #560

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

Merged
merged 5 commits into from
Oct 5, 2023

Conversation

lihebi
Copy link
Collaborator

@lihebi lihebi commented Oct 4, 2023

The goal is to build a codepod cmd tool and use like this:

> codepod /path/to/repo
# Output:
# codepod has been started, open HTTP://localhost:3000 in your browser

@lihebi lihebi mentioned this pull request Oct 4, 2023
@lihebi
Copy link
Collaborator Author

lihebi commented Oct 5, 2023

A next PR will clean up folders and leave only CMD app related code.

@lihebi
Copy link
Collaborator Author

lihebi commented Oct 5, 2023

This PR also makes two setup changes:

1. remove bun, and use back pnpm (undo #555).

The reason is that bun maintain a binary lock file, which makes no sense to me. It makes it impossible to merge PRs.

2. do not use docker anymore, for two reasons

  1. Docker isn't quite compatible with node packages (not only pnpm, but also npm/yarn for arch-related errors).
  2. Also, the source code volume mounting for development often couldn't trigger hot-reloading. As a result, I have to restart the docker stack quite often.

@lihebi lihebi merged commit 5677a41 into codepod-io:main Oct 5, 2023
@lihebi lihebi changed the title [prototype] local cmd app major refactor: on switching to a local cmd app Oct 5, 2023
@lihebi lihebi added the major label Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant