Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Applys explcit includes ruff file #773

Merged
merged 18 commits into from
Nov 12, 2024
Merged

Applys explcit includes ruff file #773

merged 18 commits into from
Nov 12, 2024

Conversation

CyclingNinja
Copy link
Member

We've selected specific .ruff.toml rule categories to apply, this is the inital test of them

@CyclingNinja
Copy link
Member Author

autofix

@CyclingNinja CyclingNinja force-pushed the refresh_ruff branch 2 times, most recently from 4d8bfb6 to 5b24785 Compare October 23, 2024 15:42
@CyclingNinja CyclingNinja force-pushed the refresh_ruff branch 7 times, most recently from c3034e4 to 4737a85 Compare October 23, 2024 19:09
nabobalis
nabobalis reviewed Oct 23, 2024
View reviewed changes
.ruff.toml Outdated Show resolved Hide resolved
nabobalis
nabobalis reviewed Oct 23, 2024
View reviewed changes
.ruff.toml Show resolved Hide resolved
nabobalis
nabobalis reviewed Oct 23, 2024
View reviewed changes
.ruff.toml Outdated Show resolved Hide resolved
nabobalis
nabobalis reviewed Oct 23, 2024
View reviewed changes
ndcube/__init__.py Outdated Show resolved Hide resolved
nabobalis
nabobalis reviewed Oct 23, 2024
View reviewed changes
.ruff.toml Outdated Show resolved Hide resolved
@CyclingNinja
Update outdate numpy
efdc978 
Update np.random.rand to np.random.Generate
per NPY002
@CyclingNinja CyclingNinja force-pushed the refresh_ruff branch 5 times, most recently from c4e2f42 to f31bf03 Compare October 24, 2024 15:50
@nabobalis nabobalis mentioned this pull request Nov 12, 2024
Closed
@nabobalis nabobalis added this to the 2.3.0 milestone Nov 12, 2024
nabobalis
nabobalis reviewed Nov 12, 2024
View reviewed changes
.pre-commit-config.yaml Outdated Show resolved Hide resolved
nabobalis
nabobalis reviewed Nov 12, 2024
View reviewed changes
.ruff.toml Outdated Show resolved Hide resolved
nabobalis
nabobalis reviewed Nov 12, 2024
View reviewed changes
.ruff.toml Outdated Show resolved Hide resolved
nabobalis
nabobalis reviewed Nov 12, 2024
View reviewed changes
ndcube/__init__.py Outdated Show resolved Hide resolved
nabobalis
nabobalis reviewed Nov 12, 2024
View reviewed changes
ndcube/tests/helpers.py Outdated Show resolved Hide resolved
Cadair
Cadair requested changes Nov 12, 2024
View reviewed changes
ndcube/_dev/scm_version.py Outdated Show resolved Hide resolved
ndcube/extra_coords/table_coord.py Outdated Show resolved Hide resolved
ndcube/extra_coords/table_coord.py Outdated Show resolved Hide resolved
ndcube/wcs/wrappers/tests/test_resampled_wcs.py Outdated Show resolved Hide resolved
@nabobalis nabobalis force-pushed the refresh_ruff branch 4 times, most recently from dbfb402 to d4b1da0 Compare November 12, 2024 16:49
@nabobalis nabobalis added the No Changelog Entry Needed Skip any changelog checks. label Nov 12, 2024
@nabobalis nabobalis force-pushed the refresh_ruff branch 3 times, most recently from d069a2b to 5b91085 Compare November 12, 2024 17:55
@nabobalis nabobalis merged commit 60a0c79 into sunpy:main Nov 12, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nabobalis nabobalis nabobalis approved these changes

@Cadair Cadair Cadair approved these changes

Assignees
No one assigned
Labels
No Changelog Entry Needed Skip any changelog checks.
Projects
None yet
Milestone
2.3.0
Development

Successfully merging this pull request may close these issues.
3 participants
@CyclingNinja @Cadair @nabobalis