astronomy-commons / astronomy-commons/lsdb

Reduce number of positional arguments

Open
#430 0 comments 0 reactions 0 assignees View on GitHub
bug
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.