abo-abo / auto-yasnippet

quickly create disposable yasnippets
245 stars 15 forks source link

Split aya-persist into aya-write-yasnippet and aya-build-yasnippet #7

Closed jasonm23 closed 9 years ago

jasonm23 commented 9 years ago

Note this is incomplete, README will need to be updated when the features are ratified.

I expect both have room to change and are separate commands in their own right.

abo-abo commented 9 years ago

There's too much copy pasting, I don't like it.

Here's how I expect you to write it:

(defun aya-write-yasnippet ()
  (interactive)
  (let ((aya-insert-snippet-function
         'aya-insert-snippet-function-extra))
    (call-interactively 'aya-persist-snippet)))

You can add as many parameters to aya-insert-snippet-function-extra as you want.

jasonm23 commented 9 years ago

Back from vacation, I'll get this organised soon.

jasonm23 commented 9 years ago

Let's kill this off, I think it's better as a separate package which depends on auto-snippet.

That way the intent of auto-yasnippet as a way to create temporary yasnippets isn't diluted.

I can also then build the feature however I feel like, since I think I'm the guy who actually wants it.

abo-abo commented 9 years ago

Good idea.