Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Package template keymap is busted #36

Closed
jbarnette opened this issue Nov 26, 2013 · 2 comments · Fixed by #37
Closed

Package template keymap is busted #36

jbarnette opened this issue Nov 26, 2013 · 2 comments · Fixed by #37
Labels
bug

Comments

@jbarnette
Copy link

https://github.com/atom/apm/blob/cdfc4ffd019c61cfece2dafb7a9a12ea01bdb520/templates/package/keymaps/__package-name__.cson.template

I think the problem is that it's using meta and it looks like keymaps are headed in a more OS-specific direction. I was going to PR something up, but I realized I had no idea what we want to teach people in this example file.
@mcolyer
Copy link
Contributor

mcolyer commented Nov 26, 2013

Yep, these should be updated. Thanks for catching that.

@mcolyer mcolyer mentioned this issue Nov 26, 2013
Merged
@mcolyer
Copy link
Contributor

mcolyer commented Nov 26, 2013

I've put together #37 which I think should fix this up. @jbarnette let me know if you think something is missing there.

DeeDeeG pushed a commit to DeeDeeG/apm that referenced this issue Jan 13, 2021
@aminya
Merge pull request atom#36 from atom-ide-community/Scripts
e779407
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Updated package template with latest changes atom/apm
2 participants
@mcolyer @jbarnette