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

refactor: @farmfe/cli #1123

Closed
ErKeLost opened this issue Apr 4, 2024 · 1 comment · May be fixed by #1517
Closed

refactor: @farmfe/cli #1123

ErKeLost opened this issue Apr 4, 2024 · 1 comment · May be fixed by #1517
Assignees
@ErKeLost
Labels
optimize performance Performance dependent refactor refactor some feature

Comments

@ErKeLost
Copy link
Member

ErKeLost commented Apr 4, 2024

What problem does this feature solve?

Use more modern tools to optimize farm cli functionality as well as functional programming and performance issues

  1. Use citty rewrite cli
  2. Split functional structure watch , build, start, preview, clean command
  3. Rewrite cli with more elegant functional programming

What does the proposed API look like?

detail see citty
@ErKeLost ErKeLost changed the title refactor: refactor @farmfe/cli refactor: @farmfe/cli Apr 4, 2024
@ErKeLost ErKeLost added optimize refactor refactor some feature performance Performance dependent labels Apr 4, 2024
@ErKeLost ErKeLost self-assigned this Apr 4, 2024
@fu050409
Copy link
Member

fu050409 commented Jul 3, 2024

Working on this issue

fu050409 added a commit to fu050409/farm that referenced this issue Jul 3, 2024
@fu050409
refactor(cli): use citty rewrite cli
f2ac316 
- Use `citty` rewrite cli
- Refactor all commands to citty
- Defaults to start a dev server

Resolved: farm-fe#1123
@fu050409 fu050409 mentioned this issue Jul 3, 2024
Draft
fu050409 added a commit to fu050409/farm that referenced this issue Jul 3, 2024
@fu050409
fix(cli): fix common args
439f3c0 
- Fix alias of common args wont be resolved by citty

Related Issue: farm-fe#1123
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ErKeLost ErKeLost

Labels
optimize performance Performance dependent refactor refactor some feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor(cli): use citty rewrite cli fu050409/farm
2 participants
@fu050409 @ErKeLost