mpickering / hlint-refactor-mode

Emacs bindings for hlint's --refactor option
MIT License
27 stars 5 forks source link

elpa #4

Closed fatlazycat closed 8 years ago

fatlazycat commented 8 years ago

Hi, is this packaged on elpa etc ?

Thanks

cocreature commented 8 years ago

It’s not yet on melpa. Feel free to submit it to melpa yourself following the instructions in the readme.

If you don’t get to it I’ll do it when I have time.

fatlazycat commented 8 years ago

Looks pretty easy will try and give it a whirl shortly.

fatlazycat commented 8 years ago

Getting Warning: (error "Package lacks a file header") when building - will look at adding one

fatlazycat commented 8 years ago

https://github.com/milkypostman/melpa/pull/3320

cocreature commented 8 years ago

thanks :+1:

fatlazycat commented 8 years ago

Rather duplication but https://github.com/milkypostman/melpa/pull/3320#issuecomment-159179284

You guys happy to make the changes ?

cocreature commented 8 years ago

I'll make the necessary changes later today.

mpickering commented 8 years ago

I would prefer not to rename it to hlint-refactor as then I think it's confusing why only the emacs bindings live here. What do people usually do?

cocreature commented 8 years ago

We can leave the repo name and add a note to the readme. This is mostly about the naming of the files and some functions, but if you’re at a point where you take a look at those, you already know that it’s an emacs mode.

cocreature commented 8 years ago

See https://github.com/milkypostman/melpa/commit/79363587b07f68e30a98698fd33ab39153bcb06c