Skip to content

Ruff format #404

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.

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Ruff format #404

wants to merge 4 commits into from

Conversation

ialarmedalien
Copy link
Contributor

Run ruff format on the codebase.

Update the main.yaml workflow to prevent tests from being run if the code QC fails.

This PR should be reviewed after #403 is merged as the first two commits are from that PR. Unfortunately I don't have write perms so can't create a branch and choose #403 to merge into.

- move tox.ini file and add config to pyproject.toml
- add codespell / code format checks to the main workflow
- add more linter checks to the linter config
Copy link

codecov bot commented Jul 6, 2025

Codecov Report

Attention: Patch coverage is 52.73973% with 276 lines in your changes missing coverage. Please review.

Project coverage is 64.45%. Comparing base (faac277) to head (7615438).

Files with missing lines Patch % Lines
linkml_runtime/linkml_model/datasets.py 0.00% 86 Missing ⚠️
linkml_runtime/linkml_model/validation.py 0.00% 26 Missing ⚠️
linkml_runtime/utils/schemaview_cli.py 0.00% 22 Missing ⚠️
linkml_runtime/linkml_model/mappings.py 0.00% 15 Missing ⚠️
linkml_runtime/utils/namespaces.py 51.85% 10 Missing and 3 partials ⚠️
linkml_runtime/processing/referencevalidator.py 72.72% 10 Missing and 2 partials ⚠️
linkml_runtime/utils/yamlutils.py 72.22% 9 Missing and 1 partial ⚠️
linkml_runtime/loaders/loader_root.py 55.55% 7 Missing and 1 partial ⚠️
linkml_runtime/processing/validation_datamodel.py 57.89% 5 Missing and 3 partials ⚠️
linkml_runtime/utils/metamodelcore.py 55.55% 7 Missing and 1 partial ⚠️
... and 18 more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #404   +/-   ##
=======================================
  Coverage   64.45%   64.45%           
=======================================
  Files          62       62           
  Lines        8846     8846           
  Branches     2556     2556           
=======================================
  Hits         5702     5702           
  Misses       2535     2535           
  Partials      609      609           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant