-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
fix: explode yaml anchors #5987
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
Conversation
* do not require code changes at several places * self referencing anchors cause error
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
|
Fixed here an additional bug in cycle detection of e.g. a: &b x
b: &a
ref: *b
c: *aNow passes, previously this was accidentally detected as cycle. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5987 +/- ##
==========================================
+ Coverage 74.65% 76.78% +2.12%
==========================================
Files 73 74 +1
Lines 11139 9225 -1914
==========================================
- Hits 8316 7083 -1233
+ Misses 2186 1505 -681
Partials 637 637 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Merge Queue Status✅ The pull request has been merged at d4d57d8 This pull request spent 6 seconds in the queue, with no time running CI. Required conditions to merge
|
|
Thank you for taking a look ❤️ |

Closes #5947