Skip to content
New issue

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

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

Already on GitHub? # to your account

chore: fix ruff configuration #186

Merged
merged 1 commit into from
Feb 1, 2023
Merged

chore: fix ruff configuration #186

merged 1 commit into from
Feb 1, 2023

Conversation

henryiii
Copy link
Collaborator

@henryiii henryiii commented Jan 31, 2023

Some of our files are being skipped because they have "build" in the name. Noticed this in pypa/build#573, and turns out problems were being masked here too.

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
@codecov
Copy link

codecov bot commented Jan 31, 2023

Codecov Report

Merging #186 (2eeaf96) into main (6003ca5) will decrease coverage by 0.14%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main     #186      +/-   ##
==========================================
- Coverage   89.18%   89.05%   -0.14%     
==========================================
  Files          44       44              
  Lines        1887     1891       +4     
==========================================
+ Hits         1683     1684       +1     
- Misses        204      207       +3     
Impacted Files Coverage Δ
src/scikit_build_core/build/wheel.py 88.98% <0.00%> (-2.33%) ⬇️
src/scikit_build_core/build/_file_processor.py 95.23% <100.00%> (+0.23%) ⬆️

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

@henryiii henryiii merged commit 4003a83 into main Feb 1, 2023
@henryiii henryiii deleted the henryiii/chore/fixruff branch February 1, 2023 02:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant