Rule action persistence policies

[rcahoon]

Description

Allow the user to specify how the RuleEngine should handle a Rule's action when the action completes or the Rule stops triggering.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Be detailed so that your code reviewer can understand exactly how much and what kinds of testing were done, and which might still be worthwhile to do.

• Unit tests: Units tests for new functionality are included in this PR
• Simulator testing: [Add your description here]
• On-robot bench testing: [Add your description here]
• On-robot field testing: [Add your description here]

[dejabot]

Thanks! I was wondering since this is only adding in the one class. Maybe that's the only one worth static imports at this time, tho..

…

-------- Original Message --------

On 12/12/24 8:43 PM, Ryan Cahoon wrote: @rcahoon commented on this pull request. --------------------------------------------------------------- In [.vscode/settings.json](#146 (comment)): > @@ -33,5 +33,17 @@ "diffEditor.ignoreTrimWhitespace": false, "java.format.settings.url": "https://raw.githubusercontent.com/google/styleguide/gh-pages/eclipse-java-google-style.xml", "java.checkstyle.configuration": "${workspaceFolder}/.github/linters/checkstyle.xml", - "java.checkstyle.version": "9.3" + "java.checkstyle.version": "9.3", + "java.completion.favoriteStaticMembers": [ + "com.team766.framework3.RulePersistence.*", Yep, this makes autocomplete include static members of this class, even if there isn't an import static for them yet (autocomplete will add the import static automatically) — Reply to this email directly, [view it on GitHub](#146 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AY4ADJVIWCB3SUE4QCZNAYT2FJQYLAVCNFSM6AAAAABTQ5NXHSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMBRGI4DONZQG4). You are receiving this because your review was requested.Message ID: ***@***.***>

[rcahoon]

Thanks! I was wondering since this is only adding in the one class. Maybe that's the only one worth static imports at this time, tho..

We can certainly add more :) I was just keeping the changes contained to those relevant to the rest of this PR