astronomy-commons / astronomy-commons/lsdb
Reduce number of positional arguments
- Dominant language
- Python
- Stars
- 55
- Forks
- 26
- Avg merge
- 4d 1h
- Merged PRs (30d)
- 8
Description
`pylint 1.0.3` started reporting that the package has an unusual number of positional arguments in some of the methods ([CI logs](https://github.com/astronomy-commons/lsdb/actions/runs/11132890320/job/30937761516#step:5:329) for reference). We should try passing them as keyword arguments whenever possible, and reduce the number of positional arguments set in the source and test `.pylintrc`.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start with the linked CI pylint output, then inspect the package source and the source and test .pylintrc files to identify the reported positional-argument cases. Done means using keyword arguments where appropriate and lowering the configured positional-argument limit without introducing new lint failures.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100