luogu-dev / luogu-dev/cyaron

feat: 对于未使用的参数报警

Open
#177 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
1.7k
Forks
182
PR merge metrics
No merged PRs in 30d

Description

例如 `repeated_edges=...` 容易记成 `repeated_edge`,而库中的函数签名设计对 IDE 也不友好。这可能导致出现用户预期外的行为。

建议将所有参数都记录在函数签名中,为了向下兼容保留 `*args` 和 `**kwargs`,并在此类参数被使用时报警。

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by auditing the library's public function signatures and locating uses of *args and **kwargs. Done means parameters are represented in signatures, legacy argument handling remains compatible, and use of unsupported or misspelled arguments produces a warning.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend-api-design
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.