Skip to content

Extend checks and add pre-commit to CI #410

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
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Extend checks and add pre-commit to CI #410

wants to merge 1 commit into from

Conversation

@vladistan
Copy link
Contributor

No description provided.

dalito
dalito reviewed Jul 18, 2025
View reviewed changes
@dalito
Copy link
Member

dalito commented Jul 18, 2025 •
edited
Loading

I'll merge #406 and we can improve with this one (like https in link to editoconfig site) and add its tighter checks.

@dalito dalito changed the title Add checks and pre commit Extend checks and add pre-commit to CI Jul 18, 2025
@vladistan vladistan force-pushed the add-checks-and-pre-commit branch 2 times, most recently from d0fa10b to 3fc6d94 Compare July 18, 2025 20:06
@vladistan
Copy link
Contributor Author

vladistan commented Jul 18, 2025 •
edited
Loading

So this one has two parts. First is fixing up config files like .editorconfig and main workflow, making them more similar to their counterparts in linkml repo. Second part is tailoring quality checks to the runtime, and also making checks be similar to linkml, like performing full pre-commit run on all files instead of just running ruff and codespell on python files.

I am thinking about splitting it into two smaller PRs. Although this PR is not particularly large. What do you think?

@vladistan vladistan force-pushed the add-checks-and-pre-commit branch from 3fc6d94 to 6110767 Compare July 18, 2025 20:51
@vladistan vladistan marked this pull request as draft July 18, 2025 20:57
@vladistan
Copy link
Contributor Author

Ok, I split this into two PRs. Turning this one into draft until we fix all problems highlighted by the lint

@vladistan vladistan force-pushed the add-checks-and-pre-commit branch 5 times, most recently from c950d43 to 8fe0d1c Compare July 22, 2025 22:57
@vladistan vladistan force-pushed the add-checks-and-pre-commit branch 4 times, most recently from 2ebe78f to 886394d Compare August 4, 2025 21:20
@vladistan vladistan force-pushed the add-checks-and-pre-commit branch from 886394d to da8647e Compare August 18, 2025 02:50
@vladistan vladistan force-pushed the add-checks-and-pre-commit branch 2 times, most recently from c31ede5 to 05dc981 Compare September 3, 2025 02:12
@vladistan vladistan force-pushed the add-checks-and-pre-commit branch from 05dc981 to 4880118 Compare October 16, 2025 01:40
@amc-corey-cox
Copy link
Contributor

@vladistan I have moved the work on this PR into the repo branch https://github.com/linkml/linkml-runtime/tree/add-checks-and-pre-commit . If you would like to continue this work please re-open the PR on the mono-repo after the merge from that branch. We'll lift it over for you so it should take less effort.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kevinschaper kevinschaper kevinschaper approved these changes

+1 more reviewer

@dalito dalito dalito left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vladistan @dalito @amc-corey-cox @kevinschaper