Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Package.getActivationCommands is deprecated. #67

Closed
CristyTarantino opened this issue May 1, 2015 · 3 comments
Closed

Package.getActivationCommands is deprecated. #67

CristyTarantino opened this issue May 1, 2015 · 3 comments

Comments

@CristyTarantino
Copy link

Use activationCommands instead of activationEvents in your package.json
Commands should be grouped by selector as follows:

  "activationCommands": {
    "atom-workspace": ["foo:bar", "foo:baz"],
    "atom-text-editor": ["foo:quux"]
  }
Package.getActivationCommands (C:\Users\tarantinom\AppData\Local\atom\app-0.196.0\resources\app.asar\src\package.js:790:9)
Package.hasActivationCommands (C:\Users\tarantinom\AppData\Local\atom\app-0.196.0\resources\app.asar\src\package.js:715:20)
<unknown> (C:\Users\tarantinom\AppData\Local\atom\app-0.196.0\resources\app.asar\src\package.js:169:24)
Package.measure (C:\Users\tarantinom\AppData\Local\atom\app-0.196.0\resources\app.asar\src\package.js:147:15)
@stramel
Copy link

stramel commented May 2, 2015

+1

d4k0 pushed a commit to d4k0/linter-jshint that referenced this issue May 3, 2015
…ter#59/AtomLinter#66, i. e. "Package.getActivationCommands is deprecated" and "Package.activateConfig is deprecated".
@d4k0
Copy link
Contributor

d4k0 commented May 3, 2015

I just created a pull request with the needed changes: #71

iam4x pushed a commit that referenced this issue May 4, 2015
This should fix #60/#67/#68 and #59/#66, i. e. "Package.getActivationCom...
@iam4x iam4x closed this as completed May 4, 2015
@Arcanemagus
Copy link
Member

Duplicate of #60.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants