-
Notifications
You must be signed in to change notification settings - Fork 21
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
SC: Add TransactionStatements functions to work with lists #849
SC: Add TransactionStatements functions to work with lists #849
Conversation
def49d5
to
3d39ad4
Compare
Latest commit added plenty of unit tests related to another branch (the one with the reduce fn). I suggest not to read the tests in |
f1d8c19
to
88cadf3
Compare
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.
Ok, I think the PR does the work.
I'm not really fan of the blacklisting, but ok for now.
We should later provide a better way with whitelisting a term and blacklisting the remaining, maybe using a dedicated parser for each function.
1038ece
to
498da8d
Compare
Description
Add TransactionStatements functions to work with lists.
Authorize Keyword lists in actions. I transformed the whitelist into a blacklist (whitelisting all keyword and blacklisted some keywords from some functions)
Part of #826
Type of change
How Has This Been Tested?
Tests added and tested with the explorer
Checklist: