Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add recipe for refs.el #4274

Merged
merged 3 commits into from Sep 29, 2016
Merged

Add recipe for refs.el #4274

merged 3 commits into from Sep 29, 2016

Conversation

Wilfred
Copy link
Contributor

@Wilfred Wilfred commented Sep 26, 2016

Brief summary of what the package does

refs.el is an Emacs package for finding references to functions, macros or variables. It works by parsing the elisp and analysing it.

refs_screenshot

Direct link to the package repository

https://github.com/Wilfred/refs.el

Your association with the package

I'm the author.

Checklist

@syohex
Copy link
Contributor

syohex commented Sep 26, 2016

  • no autoload cookie
  • I suppose :files attribute should be
    • :defaults + :exclude
    • Or move benchmark file under benchmark and use default value(= not specify)

@purcell
Copy link
Member

purcell commented Sep 26, 2016

Is it wise to use such a general name for a package which deals with only elisp refs? Why not erefs, for example?

@Wilfred
Copy link
Contributor Author

Wilfred commented Sep 26, 2016

OK, I've moved to using :defaults in the recipe.

There are already autoload cookies, have I missed any places where I should have them @syohex?

Regarding the name, I like it because I think it reads well: refs-function -- "show me the code that references this function". I may also generalise it to other languages in future.

@Wilfred
Copy link
Contributor Author

Wilfred commented Sep 29, 2016

@purcell OK, I agree. Looking at other packages, like elisp-slime-nav, it's worth clarifying.

I've rebranded the package as elisp-refs. Does everything look good now? :)

@purcell purcell merged commit 6089109 into melpa:master Sep 29, 2016
@purcell
Copy link
Member

purcell commented Sep 29, 2016

Super, thanks! Merged. :-)

@Wilfred Wilfred deleted the refs_el branch September 30, 2016 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants