galaxyproject / galaxy

Data intensive science for everyone.
https://galaxyproject.org
Other
1.42k stars 1.01k forks source link

[24.2] Fix correctness of assertion validation in new linter. #19203

Open jmchilton opened 3 days ago

jmchilton commented 3 days ago

We were using the YAML linters which were more strongly typed to validate the XML without a lot of processing - including no type conversion. Fixing this found some other problems also. Huge thanks to @bernt-matthias for reporting the issue - https://github.com/galaxyproject/galaxy/issues/19197#issuecomment-2498560642.

This just fixes the correctness. We still probably need a way to improve the pydantic output for these validation errors.

How to test the changes?

(Select all options that apply)

License