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: 给默认plan_root加入examples目录 #205

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kuainx
Copy link
Contributor

@kuainx kuainx commented Feb 14, 2025

如果脚本运行目录没有plans,判断一下有没有examples/plans目录,如果有,则使用

Copy link
Member

@KirisameVanilla KirisameVanilla left a comment

Choose a reason for hiding this comment

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

我喜欢你

@veadex
Copy link
Contributor

veadex commented Feb 14, 2025

这个有什么作用吗?如果实在不想和plans放在同级的话可以指定路径

@KirisameVanilla
Copy link
Member

这个有什么作用吗?如果实在不想和plans放在同级的话可以指定路径

作用大概在不用指定路径

@kuainx
Copy link
Contributor Author

kuainx commented Feb 14, 2025

这个有什么作用吗?如果实在不想和plans放在同级的话可以指定路径

目前的路径获取逻辑是os.getcwd,这要求工作目录在examples下,所以用python examples/event.py运行时会找不到路径
本来是想判断user_settings.yaml文件路径,但是看了一下需要改的比较多
在曾经的逻辑下,会使用包内plans,但是现在都挪到了examples,所以想写个examples的目录加进去
仅对本地安装且工作目录不在examples的情况生效
不想和plans放在同级的原因:个人想法尽量少改动配置文件:(

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.

3 participants