astropy / astropy/astroquery

Consider using pyupgrade to notify for old idioms

Open
#2,410 1 comment 0 reactions 0 assignees View on GitHub
cleanup infrastructure
Dominant language
Python
Stars
791
Forks
451
Avg merge
1d 3h
Merged PRs (30d)
4

Description

I notice during code reviews that a lot of the PRs use old python syntax that are not required any more, or even better idioms that we have already cleaned out from the codebase before. Some of these slip through, which is not ideal.

Using tools like https://github.com/asottile/pyupgrade e.g. in CI or as a pre-commit hook could resolve this issue.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the repository's current CI and pre-commit configuration, then assess how pyupgrade could report old Python syntax or idioms during code review. Done means the selected integration is configured, produces useful notifications, and does not introduce unwanted failures for existing code.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
ci-cd, tooling
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.