Yaml parser (New)#2383
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #2383 +/- ##
==========================================
- Coverage 58.19% 57.21% -0.99%
==========================================
Files 475 324 -151
Lines 47819 38931 -8888
Branches 8509 7126 -1383
==========================================
- Hits 27828 22273 -5555
+ Misses 19106 15898 -3208
+ Partials 885 760 -125
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Hook25
force-pushed
the
yaml_parser
branch
6 times, most recently
from
ecceb91 to
8717e89
Compare
fernando79513
left a comment
fernando79513
left a comment
There was a problem hiding this comment.
Great job!
If at some point we want to hard deprecate the .pxu files and clean the code, it's not going to be a super easy task, but at least all the markers you are putting for the comments will help.
Thanks for also taking care of handling properly the suspend flags, and especially the get_array_field_qualify and all the Visitors.
I run the smoke tests from the yaml and they passed no problem.
Let's see if the translated metabox tests also pass in the CI.
|
I manually ran some YAML jobs and test/plans, and they worked fine. I think any other issues that may have slipped through should be raised during metabox tests. |
pieqq
left a comment
pieqq
left a comment
There was a problem hiding this comment.
Huge undertaking, good job on this!
I spotted one weird thing with some function names, and I left a few comments.
This didnt work before because it was also forced translatable and when that was the case the forced variant check was skipped (sigh). This removes both
Minor: use list comprehension to parse resource
Minor: raise an exception instead of assert
flags weren't correctly treatedfor manual after suspend jobs because I didn't notice them. Now the code is unique so it is easier to make changes for both
3 participants
Description
PXUs are a very outdated and weird format that scares off new contributors and makes the friction to contribute to Checkbox higher.
This PR introduces a new supported format: YAML
To begin with this transition, this translates all the pxus in Checkbox (plainbox provider) + all the pxus in metabox. The translation of the rest of the providers will be done and tested gradually in followup PRs.
Resolved issues
Fixes: CHECKBOX-1392
Documentation
N/A (Pending)
Tests
Unit + integration tested most of the additions