-
Notifications
You must be signed in to change notification settings - Fork 35
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.
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
base: main
Are you sure you want to change the base?
Conversation
I'll merge #406 and we can improve with this one (like https in link to editoconfig site) and add its tighter checks. |
d0fa10b
to
3fc6d94
Compare
So this one has two parts. First is fixing up config files like I am thinking about splitting it into two smaller PRs. Although this PR is not particularly large. What do you think? |
3fc6d94
to
6110767
Compare
Ok, I split this into two PRs. Turning this one into draft until we fix all problems highlighted by the lint |
c950d43
to
8fe0d1c
Compare
2ebe78f
to
886394d
Compare
886394d
to
da8647e
Compare
c31ede5
to
05dc981
Compare
05dc981
to
4880118
Compare
@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. |
No description provided.