lanl / dsi

LANL Data Science Infrastructure Project
https://lanl.github.io/dsi
5 stars 3 forks source link

Fixing some merge related bugs #69

Closed qwofford closed 1 year ago

qwofford commented 1 year ago

When I merged the SystemKernel plugin serializer PR, and then the type checking PR, I stomped on some logic due to an incompatibility between these two PRs. In the future, the correct order of operations is to merge one PR into main, then merge main into the second PR and begin testing again before merging the second PR.

Please review this branch to ensure both the SystemKernel Serializer and the pydantic type checking capability are intact.