-
Notifications
You must be signed in to change notification settings - Fork 294
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
Support circular references in tags and better array formatting #659
Conversation
|
Codecov Report
@@ Coverage Diff @@
## master #659 +/- ##
==========================================
+ Coverage 96.22% 96.23% +<.01%
==========================================
Files 94 94
Lines 3234 3240 +6
==========================================
+ Hits 3112 3118 +6
Misses 122 122
Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #659 +/- ##
==========================================
+ Coverage 96.22% 96.23% +<.01%
==========================================
Files 94 94
Lines 3234 3240 +6
==========================================
+ Hits 3112 3118 +6
Misses 122 122
Continue to review full report at Codecov.
|
What does this PR do?
This PR removes depth when it comes to tags and instead detects circular references. This will allow us to get infinite visibility for objects-as-tags.
This PR also changes the way we format arrays. We now leverage
JSON.stringify
to nest arrays rather than flattening them.Motivation
We want to ensure we can always get meaningful information from tags while at the same time ensuring readability.
Additional Notes
Please check the tests to see what the expected behaviour is, thanks!