Skip to content

Conversation

@clark874
Copy link

@clark874 clark874 commented Jan 1, 2025

  1. 添加配置文件管理系统
  2. 添加智能分析工具
  3. 优化输出路径管理

1. 添加配置文件管理系统
2. 添加智能分析工具
3. 优化输出路径管理
@filipchristiansen
Copy link
Contributor

Add intelligent code repository analysis tool

  1. Add configuration file management system
  2. Add intelligent analysis tool
  3. Optimize output path management

(English)

@cyclotruc cyclotruc changed the title feat: 添加智能代码仓库分析工具 feat: Add intelligent code repository analysis tool Jan 2, 2025
@cyclotruc cyclotruc added the changes requested A review requested changes before merging label Jan 2, 2025
Copy link
Member

@cyclotruc cyclotruc left a comment

Choose a reason for hiding this comment

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

@clark874 Hello and thank you for your contribution

Could you explain a bit what your changes are trying to achieve and how I can test them?
It seems like you are doing a new CLI out of smart_ingest.py but we already have structure for a CLI in cli.py that we should re-use here

Also, we stick with english for the entire repo, but we will be happy to translate your changes once they are ready (like @filipchristiansen kindly did in #95)

(用人工智能翻译):
你好,感谢你的贡献!
你能解释一下你的修改想要达到什么目的,以及我该如何测试它们吗?
看起来你正在用 smart_ingest.py 创建一个新的命令行界面,但我们在 cli.py 中已经有了命令行界面的结构,我们应该在这里重用它。
另外,我们整个代码仓库都使用英语,但是一旦你的修改准备就绪,我们很乐意翻译你的修改(就像 @filipchristiansen#95 中友善地做的那样)。

@cyclotruc
Copy link
Member

@clark874 I'm closing this for now but if you ever want to pursue on this I'd be happy to open it again

@cyclotruc cyclotruc closed this Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changes requested A review requested changes before merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants