i alarmed alien
i alarmed alien
I was going to put in a PR with some ruff formatting/linting, but since this one is already done, it'd be much better to use this instead. Could you update...
See also the recently-implemented ruff formatting and linting in [linkml-map](https://github.com/linkml/linkml-map/).
@Silvanoc I mostly took the path of least change (least resistance) with the linkml-map set up but would prefer to have a standardised formatting/linting set up in as many of...
[This PR](https://github.com/linkml/linkml/pull/2648) fixes the upstream test issues but it's awaiting the release of linkml-runtime. Gotta love tightly-coupled packages!
Do we need black and ruff as code formatters? Due to the amount of work involved in generating this PR, I would be inclined to merge as-is and make a...
@Silvanoc > do I understand it correctly? The conversations that you've opened in this PR are mostly justifying some of the changes, right? So a reviewer that holds those changes...
@Silvanoc re: ruff linting: I think it would be better if you made a direct PR to the linkml-runtime repo to update ruff and add it to the dev dependencies...
> IMO the whole linting, ruff, tox,... set-up in both `linkml` (especially there) and `linkml-runtime` is a mess. I would seriously consider cleaning it up... I agree completely! I did...
@Silvanoc Apologies for unintentionally giving you a load of detective work to do in figuring out my ruff config! I would have included it as a comment on the PR...
Migration of linkml-map to uv: https://github.com/linkml/linkml-map/pull/54 -- reviewers welcome!