VorTECHsa / VorTECHsa/python-sdk
Reference objects argument `id` shadows python built-in
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 25
- Forks
- 12
- Avg merge
- 20h 47m
- Merged PRs (30d)
- 2
Description
id is a built-in in python - worth considering whether we should change id argument of Reference class to something like vortexa_id
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the Reference class and search the SDK for its id argument and call sites. Check how the public argument is used and whether existing tests cover it; done means the naming no longer shadows Python's built-in without breaking supported usages.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- api
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100