TheAlgorithms / TheAlgorithms/Python

Should we add matchify to pre-commit

未关闭
#15,337 8 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 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?

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 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

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。