-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Added support for dereferenced map properties #2635
Conversation
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.
Thank you for implementing this 👍
I may cherry pick this into my new upstream, on actions-oss to release this earlier
Co-authored-by: ChristopherHX <christopher.homberger@web.de>
Co-authored-by: ChristopherHX <christopher.homberger@web.de>
@ChristopherHX thank you for this awesome tool! I've applied your suggestions so feel free to merge at any time :) |
@m1r4c this pull request has failed checks 🛠 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2635 +/- ##
==========================================
+ Coverage 61.56% 68.72% +7.16%
==========================================
Files 53 71 +18
Lines 9002 10942 +1940
==========================================
+ Hits 5542 7520 +1978
+ Misses 3020 2857 -163
- Partials 440 565 +125 ☔ View full report in Codecov by Sentry. |
Fixes #2591