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

Apropos special forms #2012

Merged
merged 1 commit into from
Jun 17, 2017
Merged

Conversation

xiongtx
Copy link
Member

@xiongtx xiongtx commented Jun 16, 2017

These changes are necessary to use clojure-emacs/cider-nrepl#409.

Example of special forms from cider-apropos:

image

"Look up the grimoire documentation for SYMBOL."
"Look up the grimoire documentation for SYMBOL.

If SYMBOL is a special form, the `clojure.core' ns is used, as is
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't quote clojure.core here - this is just for Elisp symbols.

@bbatsov
Copy link
Member

bbatsov commented Jun 16, 2017

You should also add a changelog entry.

Add `apropos-special-form` button type

Default to `clojure.core` ns for Grimoire; this is necessary for special forms,
which do not have ns. Grimoire defaults to `clojure.core`.
@xiongtx
Copy link
Member Author

xiongtx commented Jun 16, 2017

What's your general rule for releasing new version of cider-nrepl and CIDER? After some time or some number of commits?

@bbatsov bbatsov merged commit 3d6b97b into clojure-emacs:master Jun 17, 2017
@bbatsov
Copy link
Member

bbatsov commented Jun 17, 2017

What's your general rule for releasing new version of cider-nrepl and CIDER? After some time or some number of commits?

There are no special rules - I release new versions when it seems to me that this is warranted. Usually I aim to cut major releases around major Clojure conferences.

@xiongtx xiongtx deleted the apropos-special-forms branch June 17, 2017 06:26
xiongtx added a commit to clojure-emacs/helm-cider that referenced this pull request Jun 17, 2017
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.

2 participants