-
Notifications
You must be signed in to change notification settings - Fork 72
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
core: helpers for IRDL construct type access. #3025
Conversation
Co-authored-by: Sasha Lopoukhine <[email protected]>
Co-authored-by: Sasha Lopoukhine <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3025 +/- ##
=======================================
Coverage 89.85% 89.86%
=======================================
Files 412 412
Lines 51823 51834 +11
Branches 8033 8004 -29
=======================================
+ Hits 46565 46579 +14
+ Misses 3967 3966 -1
+ Partials 1291 1289 -2 ☔ View full report in Codecov by Sentry. |
class VarOpResult(tuple[OpResult, ...]): | ||
@property | ||
def types(self): | ||
return tuple(r.type for r in self) |
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.
this is great if this doesn't need an or [[]]
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.
This wasn't actually needed where you used it 🙂 Happy to discuss it in chat if you're curious
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.
I like it
Implement simple type access helpers, but for IRDL