-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Correct Q+A about state #8918
Correct Q+A about state #8918
Conversation
For influxdata#8907 . After influxdata#8447 , this common question in the Starlark Processor README.md should be updated to reflect the newly available shared state.
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.
🤝 ✅ CLA has been signed. Thank you!
same as the nightly build: and other recent builds:
Probably because of #8844 "Move windows tests to circle-ci, remove appveyor". As was done in #8414 for other plugins, maybe adding |
You're right, gonna see if I can fix that |
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.
Looks good to me.
(cherry picked from commit 1b7f445)
(cherry picked from commit 1b7f445)
Closes #8907 . After #8447 , this common question in the Starlark Processor README.md should be updated to reflect the newly available shared state.