Refactor ValidationError

[eagr]

closes #660
closes #661
closes #733

[mrcnski]

Woah, awesome! I totally missed this. Is it ready for review @eagr ?

[eagr]

Not ready yet. I plan to continue on this after the artifact pr. :)

[mrcnski]

Hey @eagr, quick update from me! Getting #1685 merged is my highest priority right now, and as part of that PR I need to do a somewhat big error refactor. There hopefully won't be too many conflicts with this work, but it's pretty substantial, so consider this a heads-up! I apologize for the delays and any inconvenience, but after that PR goes in I'll be much more free to focus on other things, like reviewing the artifacts PR to unblock this one. :)

[eagr]

Isn't having merge conflicts the norm? :) Totally understand. And usually I wouldn't fixate on something. I'll just move on to the next thing, like green threads do, so no worries man.

[paritytech-cicd-pr]

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-clippy
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4230235

[eagr]

Gonna redo this based on master as it's probably faster and easier to get right than trying to solve the conflicts. :) @mrcnski