-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fieldref support #137
Fieldref support #137
Conversation
Pull Request Test Coverage Report for Build 8660522583Details
💛 - Coveralls |
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.
Nice implementation, but I noticed a couple of issues that could be problematic in edge cases :)
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.
Really minor bits of me being opinionated about Python (sorry! Feel free to push back) and a minor documentation issue - but this is otherwise a really great implementation 👍 Love the extra tests for the pipelines 🥳
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!
Adds in support for
fieldref
modifiers as part of pySigma to enable reference to other fields in the log line