TheAlgorithms / TheAlgorithms/Python
Should we add matchify to pre-commit
未关闭
还没有人认领这个 Issue。
enhancement
- 主要语言
- Python
- 星标
- 225k
- 派生
- 51.1k
- 平均合并
- 1 天 23 小时
- 30 天内合并 PR
- 156
描述
Feature description
@priya-sundaram-dev, please evaluate if we should add https://github.com/15r10nk/matchify to our codebase.
Python's match statement is powerful and we should encourage contributors to use it where it makes sense.
On Python 14t, please run uv tool install matchify && matchify data_structures, which should modify six files.
Do these changes look correct and useful? Do you recommend we add this tool to pre-commit?
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
按要求在 Python 14t 上运行 uv tool install matchify && matchify data_structures,然后检查六个已修改的文件。评估这些转换对于此代码库是否正确且有用,以及 matchify 是否应加入 pre-commit。完成的标准是记录调查结果并给出明确的建议。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- python
- 领域
- tooling
- Issue 类型
- 功能
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 活跃度
- 活跃
- 描述清晰度
- 基本清楚
- 新手友好度
- 45/100