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

feat: add options for proxy and host #1

Closed
wants to merge 3 commits into from

Conversation

fxliang
Copy link
Contributor

@fxliang fxliang commented Mar 4, 2025

No description provided.

@fxliang
Copy link
Contributor Author

fxliang commented Mar 5, 2025

用env是最无痛传递全局参数的方式,否则从接口传递进去要很长的路径相当麻烦

后面想了下,其实确实没有必要去负担proxy 或者 repo host写错的情况,直接放过不检查regex匹配就行

限制在download和install里改成函数调用

用全局参数以便这些选项可以在处理子命令之前处理了,这样就可以放前或者放后都可以

应该算是解完了

@lotem
Copy link
Owner

lotem commented Mar 9, 2025

用環境變量不行。我開始暴改。

@lotem
Copy link
Owner

lotem commented Mar 9, 2025

改完代碼,我發現設置 https_proxy 環境變量就可以了,我自己也是靠環境變量用代理的,那麼 --proxy 參數還有用嗎?

@lotem lotem closed this in 99a6e38 Mar 9, 2025
lotem added a commit that referenced this pull request Mar 9, 2025
Closes #1

Squashed commit of the following:

commit ae020da
Author: fxliang <fx.liang@outlook.com>
Date:   Tue Mar 4 20:33:00 2025 +0800

    simplify setup process of proxy and repo host

commit cfaa1e3
Author: fxliang <fx.liang@outlook.com>
Date:   Tue Mar 4 12:38:00 2025 +0800

    feat: set repo host with option `--host <host>`

commit debfddb
Author: fxliang <fx.liang@outlook.com>
Date:   Tue Mar 4 12:31:00 2025 +0800

    feat: set proxy with option `--proxy <proxy>`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants