-
Notifications
You must be signed in to change notification settings - Fork 12
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
Improve docs #356
Improve docs #356
Conversation
@Soph1514 Would you like to help with this PR? It could be a great collaboration practice before we start working on rule engine! |
@@ -143,7 +261,7 @@ fn apply_all_rules<'a>( | |||
) -> Vec<RuleResult<'a>> { | |||
let mut results = Vec::new(); | |||
for rule in rules { | |||
match rule.apply(expression, model) { |
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.
Accidentally changed this line
Commented out the docstring examples as they were causing the cargo test --workspace command to fail in the CI pipeline. The examples were meant to provide additional context for abstract functions but aren't critical to the descriptions. Open to feedback on whether to rework these examples to pass the tests or move them to a separate file for better context without affecting the CI process. @ozgurakgun |
Improving documentation. Ideally, I would like to see all methods and methods` parametres in rule_engine documented.
TODO for rewrite.rs: