You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This method became obsolete after recent changes to the supervision mechanism.
We use it outside of govnr only in one case where we want to bypass the supervision mechanism. near it, this comment appears: //TODO use real supervision?
In this case, we should either conform to the supervision semantics or stop using govnr altogether, or otherwise replace this method with a clearer ForeverNoShutdownSupervision() (which will not return a ForeverHandler but only a channel which is closed upon termination instead), or rename this workaround method so it's clear that it does the opposite of what it current name suggests: either MarkUnsupervised() to RunForeverWithoutSupervision()
Once the method is deprecated it should be marked as in internal function to govnr and not exposed to consuming modules
The text was updated successfully, but these errors were encountered:
This method became obsolete after recent changes to the supervision mechanism.
We use it outside of govnr only in one case where we want to bypass the supervision mechanism. near it, this comment appears:
//TODO use real supervision?
In this case, we should either conform to the supervision semantics or stop using govnr altogether, or otherwise replace this method with a clearer
ForeverNoShutdownSupervision()
(which will not return aForeverHandler
but only a channel which is closed upon termination instead), or rename this workaround method so it's clear that it does the opposite of what it current name suggests: eitherMarkUnsupervised()
toRunForeverWithoutSupervision()
Once the method is deprecated it should be marked as in internal function to govnr and not exposed to consuming modules
The text was updated successfully, but these errors were encountered: