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

Feature: 支持 pip 直接安装 #106

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ttimasdf
Copy link

优点:

  1. 用户安装更简单: pip install neoreg
  2. 使用更简单:从任何路径下直接运行 neoreg 命令

缺点:

  1. 需要引入 main 函数,不能像之前一样,把局部变量当全局变量不加限制的用
  2. 依赖版本和python版本,通过项目文件进行了限制,考虑到以前没有限制也没啥问题,版本限制可以继续放宽

以本 PR 的代码为例,进行测试安装:

使用 pip

pip install git+https://github.com/ttimasdf/Neo-reGeorg@feature-python-package

使用 uv

uv pip install git+https://github.com/ttimasdf/Neo-reGeorg@feature-python-package

使用 uv 全局安装(类似 pipx)

uv tool install --python 3.10 git+https://github.com/ttimasdf/Neo-reGeorg@feature-python-package

@L-codes
Copy link
Owner

L-codes commented Feb 17, 2025

首先感谢你的PR, 可是原本的安装难度也不高,单py文件运行,上面提到的缺点倒不是什么问题,我觉得比较大问题是提高了维护成本,我觉得可以继续保留其简易性

不过这个PR可以保留看看,看用户的反馈,要是支持的人多,我会再考虑合并的:)

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