Fix subs is not defined

[sertonix]

Since Atom was killed the community came together and made a fork called Pulsar.
It has updated a lot of parts in the editor.

While most packages still work some old javascript styles are not valid anymore.
This fixes a syntax error.

It would be nice if a new version with this fix could be published to the new packend. If there are any problems you can create an issue on the github repo.

[goyalyashpal]

@pfitzseb @MikeInnes can u please take a look at it 😃

[goyalyashpal]

a better change, again, as suggested by @savetheclocktower at pulsar-edit/pulsar#482 (comment) is to change these 2 instances of subs to this.subs in both these files. so:

--- a/lib/linter/pane.js
+++ b/lib/linter/pane.js
@@ -11,11 +11,11 @@
   name = 'LinterPane'
 
   static activate () {
-    subs = new CompositeDisposable()
+    this.subs = new CompositeDisposable()
   }
 
   static deactivate () {
-    subs.dispose()
+    this.subs.dispose()
   }
 
   constructor (opts) {

--- a/lib/linter/compiled-pane.js
+++ b/lib/linter/compiled-pane.js
@@ -12,11 +12,11 @@
   name = 'CompiledPane'
 
   static activate () {
-    subs = new CompositeDisposable()
+    this.subs = new CompositeDisposable()
   }
 
   static deactivate () {
-    subs.dispose()
+    this.subs.dispose()
   }
 
   constructor (opts) {

[goyalyashpal]

@sertonix ping 😃

[goyalyashpal]

is this acceptable now @pfitzseb 😃 ?

[goyalyashpal]

Remaining steps:

• Creating new tag with fixes
• Publishing new update to the packend or whatever that's called
• Modify the uber-juno package to use this fixed version (though i suppose it would use the latest version itself?)