Skip to content

chore: use ruff#175

Merged
henryiii merged 4 commits intomainfrom
henryiii/chore/ruff
Jan 21, 2023
Merged

chore: use ruff#175
henryiii merged 4 commits intomainfrom
henryiii/chore/ruff

Conversation

@henryiii
Copy link
Collaborator

@henryiii henryiii commented Jan 20, 2023

Trying this out. Very impressed so far. Some minor issues, but very complete, fast, and simple. No more .flake8 file!

@henryiii henryiii force-pushed the henryiii/chore/ruff branch from 69882e7 to 540a231 Compare January 20, 2023 01:05

if session.posargs:
if "--serve" in session.posargs:
print("Launching docs at http://localhost:8000/ - use Ctrl-C to quit")
Copy link
Collaborator Author

@henryiii henryiii Jan 20, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, wow, auto fixing no print statements might be something to turn off. But that's easy to do in Ruff. :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Huh that seems a bit aggressive!

version.source = "vcs"
build.hooks.vcs.version-file = "src/scikit_build_core/_version.py"

[tool.hatch.build.force-include]
Copy link
Collaborator Author

@henryiii henryiii Jan 20, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is working around a bug in the TOML crate for Rust, see astral-sh/ruff#1894.

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
@henryiii henryiii force-pushed the henryiii/chore/ruff branch from 540a231 to c225d32 Compare January 20, 2023 01:29
@codecov
Copy link

codecov bot commented Jan 20, 2023

Codecov Report

Merging #175 (9bff6e5) into main (c153a4c) will increase coverage by 0.15%.
The diff coverage is 67.85%.

@@            Coverage Diff             @@
##             main     #175      +/-   ##
==========================================
+ Coverage   90.13%   90.29%   +0.15%     
==========================================
  Files          44       44              
  Lines        1855     1844      -11     
==========================================
- Hits         1672     1665       -7     
+ Misses        183      179       -4     
Impacted Files Coverage Δ
src/scikit_build_core/build/__init__.py 71.42% <0.00%> (ø)
src/scikit_build_core/build/sdist.py 98.33% <ø> (ø)
src/scikit_build_core/file_api/reply.py 84.93% <0.00%> (ø)
src/scikit_build_core/setuptools/build_meta.py 77.77% <0.00%> (ø)
src/scikit_build_core/build/wheel.py 91.22% <50.00%> (ø)
src/scikit_build_core/builder/builder.py 95.45% <50.00%> (-0.06%) ⬇️
src/scikit_build_core/setuptools/extension.py 92.10% <50.00%> (-0.11%) ⬇️
src/scikit_build_core/builder/sysconfig.py 83.33% <60.00%> (-0.23%) ⬇️
src/scikit_build_core/settings/sources.py 89.81% <83.33%> (+0.96%) ⬆️
src/scikit_build_core/build/_file_processor.py 95.00% <100.00%> (-0.24%) ⬇️
... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@henryiii
Copy link
Collaborator Author

pre-commit.ci autofix

@henryiii henryiii merged commit 4d1c4f8 into main Jan 21, 2023
@henryiii henryiii deleted the henryiii/chore/ruff branch January 21, 2023 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants