Skip to content
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

RHS captures all s-expressions using implicit do #119

Merged
merged 1 commit into from
Aug 21, 2015

Conversation

jonsmock
Copy link
Contributor

Regarding issue #118

@jonsmock
Copy link
Contributor Author

This was the smallest change I could make to get the desired functionality. Probably better would be to simply return the original captured collection as rhs (and adjust defrule to check if it's empty instead of checking for nil), but doing this caused some problems in parse-rule*, so I backed it out. Let me know if you want me to make that change, and I can take another shot at it. I wanted to at least get this to you for review, and I just had my lunch break to do it.

@rbrush
Copy link
Contributor

rbrush commented Aug 21, 2015

Yes, I think the minimal approach is the way to go here. We could update parse-rule*, but that's used elsewhere in test code and in a (not publicly available) library that generates rules.

rbrush added a commit that referenced this pull request Aug 21, 2015
RHS captures all s-expressions using implicit `do`
@rbrush rbrush merged commit f095a32 into oracle-samples:master Aug 21, 2015
@rbrush
Copy link
Contributor

rbrush commented Aug 21, 2015

I went ahead and merged this. Thanks for the contribution! We can probably do a fix release with this (and a couple other recent fixes) if desired.

@jonsmock
Copy link
Contributor Author

No problem - thanks for the lib! No rush on the release. We're happily using it as is; I just figured I'd ask/fix it, if ya'll were interested. Seriously, rule engines rock. How did I not know this?! ;-)

@rbrush rbrush added this to the 0.8.9 milestone Sep 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants