|
Post by nschellenberger on Feb 9, 2023 20:35:14 GMT
Hi Folks, I realise I will seem like a nutjob for even bothering to reporting these. I found them entirely by accident while hacking on some code generation tools, and I figured I might as well. I happened to have a couple of arbitrary, ad-hoc checks for various weird things while I was debugging my generator. Two categories that Redfish (and Swordfish) tripped over were: Patches are relative to DSP8010_2022.3 and Swordfish_v1.2.4a_Schema. Clearly they make no functional difference to anything other than to a tool validating very closely.... Regards, Neil
|
|
|
Post by jautor on Feb 10, 2023 1:10:26 GMT
No, Neil, I think you're fitting right in with the rest of us...
(I say that as we've addressed several issues regarding white space and non-printable characters just recently... So "nits" that actually show up as printed characters is well, well above our cut line!)
I'm looking through your patch list, and at least some of those are obvious errors that we need to correct. I'll discuss with the group to look at the full list and report back if there's portions that we either can't fix, won't fix, or have more questions about...
Jeff
|
|
|
Post by mraineri on Feb 10, 2023 2:35:27 GMT
Neil, do you mind opening a GitHub issue in this repo? github.com/DMTF/Redfish-ToolsThese sorts of things should be caught by the "odata_validator.py" tool we have there. Thanks for pointing this out. I'm glad this is incredibly minor and doesn't impact how we really do things from a functional perspective.
|
|
|
Post by nschellenberger on Feb 10, 2023 18:27:41 GMT
|
|
|
Post by Richelle Ahlvers on Feb 14, 2023 2:37:57 GMT
Also logged in Swordfish, will confirm that odata_validator changes work across the board.
|
|