- Notifications
You must be signed in to change notification settings - Fork778
feat: add _onStart precondition for script execution#899
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
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
FYI, I have been working on this, but it will take time before it is merged. |
@facontidavide I accidentally closed this PR, so I've reintroduced it(#932) and not only added _onStart, but also handled the CI error along the way. If the _onStart feature isn't merged so soon, do I need to raise a separate PR that handles CI errors? |
Add a new precondition type _onStart that allows executing scripts before a node starts its tick. This provides symmetry with the existing _post functionality.
Resolves#895