-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RCAL-695 generalize attribute assignment and creation #284
RCAL-695 generalize attribute assignment and creation #284
Conversation
907777a
to
ffedf69
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #284 +/- ##
==========================================
+ Coverage 96.98% 97.04% +0.06%
==========================================
Files 29 28 -1
Lines 2485 2504 +19
==========================================
+ Hits 2410 2430 +20
+ Misses 75 74 -1 ☔ View full report in Codecov by Sentry. |
5140b85
to
a227a31
Compare
Can you add a PLWishMaster link to passed RCAL regression tests with this? |
I ran the regression tests here: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A follow on question to the requested change. Can this be done directly on a DataModel
object itself rather than the node
object(s) it wraps?
4ca4108
to
2a5c47c
Compare
842cc52
to
e53431f
Compare
Resolves RCAL-695
Closes #54
This PR addresses allows new node attributes to be created using dot notation so long as the attribute is consistent with the schema (e.g., an explicit property or matches a patternProperty).
It also enables validation of the created attribute's value, as well as validating object instances assigned to existing attributes.
Checklist
CHANGES.rst
under the corresponding subsection