atom/flight-manual.atom.io

Deactivation advice is incorrect

Closed this issue · 0 comments

The current advice on deactivation in the word-count page reads as follows:

This optional method is called when the window is shutting down. If your package is watching any files or holding external resources in any other way, release them here. If you're just subscribing to things on window, you don't need to worry because that's getting torn down anyway.`

This advice is incorrect, as deactivation also occurs when the package is disabled, which can happen independently of window shutdown. You should in fact dispose of all subscriptions you're holding on to.