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

Make it possible to use nested/shared sub-rules inside context #49

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

mahagr
Copy link

@mahagr mahagr commented Dec 16, 2016

NOT READY TO BE MERGED as the code isn't fully tested and there are no test cases or documentation for it yet.

This feature allows you to create sub-rules as variables and use those to build new rules. Those sub-rules are only compiled once (into true/false) and are very useful if you have a large set of different rules that get used in multiple locations.

.gitignore Outdated
@@ -1,2 +1,3 @@
vendor
composer.lock
.idea

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Personally, I feel like this should be handled by a user's global git ignore rather than the project's.

@mahagr
Copy link
Author

mahagr commented Dec 20, 2016

Added description.

Sorry about this pull request. I'm not sure why github created a pull request on my changes as I've not yet even fully tested the code. I am 100% sure that I didn't create pull request yet. Maybe I accidentally had wrong upstream...

Are you still maintaining this project? I have also some other interesting things for Ruler, like a simple compiler that compiles lisp-like language into a set of rules (php code atm).

@druid628
Copy link

No, I'm not the package maintainer, just an avid user of Ruler.

@mahagr
Copy link
Author

mahagr commented Dec 22, 2016

Oops, I just assumed you were.

Has there been any word from @bobthecow ? He has been quite inactive on this project; should we start maintaining our own version of the library or is he still interested on maintaining it?

@ludicruz
Copy link
Contributor

@bobthecow do you exist? There are quite a few PRs open including #37 which has been open for 2 years. There were a few things implemented because of the strict adherence to PHP 5.3 because it was the GA release at the time. Because of that, a good refactor and clean up would be nice to use some of the newer language features (only if warranted of course :) ).

@mahagr mahagr changed the title Feature/variable method Make it possible to use nested/shared sub-rules inside context Aug 2, 2018
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.

4 participants