Add a settings view where one can disable snippets for certain packages

[dralletje]

I know it does look kind of hacky, but this is the way it should be done I think. I am yet to dive deeper so we can reload without restarting it, maybe be just fully reloading all snippets. Made this because of this issue #55

[PierBover]

Hey @dralletje

I'm in the process of working in another PR for this repo and you will need to add tests (specs) to your fork.

• https://github.com/dralletje/snippets/tree/master/spec
• https://atom.io/docs/latest/hacking-atom-writing-specs

I'm not part of the atom team btw... but I'm undergoing this process which is completely new to me.

[aryzing]

Is there anything preventing the merge?

[winstliu]

Yes, it needs specs.

[dralletje]

Hey @50Wliu (not sure if I need to ping you), but is this enough spec that I added? 😁
I know it is only one test, I'm not very good with tests yet

[krabbypattified]

Eagerly awaiting this feature.. any updates?

[kvsm]

Just popping in to say that it's 2017 and this default snippet malarkey is still the thing which annoys me most frequently in life. Pretty please can this be reviewed (and merged). 😄

[lee-dohm]

The concern I have about this is that you have to restart for things to take effect. As the PR currently stands, I am 👎 on merging because of this. The system that was used to disable keybindings doesn't have this problem. Perhaps you can model this ability on that PR? atom/atom#8130

[dralletje]

Thanks for the feedback, @lee-dohm , I share the concern.
Let me come back to this later this week with a restart-less version :-)

[lee-dohm]

Sounds good! Thanks very much for your patience and effort on this @dralletje 🙇 ✨

[NanerLee]

so long time...
Eagerly awaiting this feature

[lee-dohm]

@NanerLee If you would like to see this happen, perhaps you could take a stab at improving the PR?

[layonferreira]

Hi, I have added the ability to reload snippets without the need to restart on #242

[dralletje]

@layonferreira Looks awesome! Thanks for picking up where I forgot hahaha 😁 First contribution including a test, good job :)

[dralletje]

Closing in favor of #242