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

Mark many custom variables as safe #2027

Merged
merged 1 commit into from
Jul 5, 2017

Conversation

SevereOverfl0w
Copy link
Contributor

@SevereOverfl0w SevereOverfl0w commented Jul 5, 2017

Based on slack discussion. These variables are useful to set in a .dir-locals.el for projects.


  • The commits are consistent with our [contribution guidelines][1]
  • All tests are passing (make test) @arichiardi
  • The new code is not generating bytecode or M-x checkdoc warnings @arichiardi
  • You've updated the changelog (if adding/changing user-visible functionality) TODO: Once PR is assigned a #

@arichiardi
Copy link
Contributor

Uhm, maybe the tests were already broken. The changes are so trivial...like this one

(defcustom cider-doc-xref-regexp "`\\(.*?\\)`"

CHANGELOG.md Outdated
@@ -34,6 +34,7 @@
* [#1945](https://github.com/clojure-emacs/cider/pull/1945): Start nREPL servers bound to `::` by default using `cider-jack-in`.
* Renamed `cider-prompt-save-file-on-load` to `cider-save-file-on-load` and adjust its supported values accordingly (the default now is `'prompt` and `'always-save` is now simply `t`).
* [#2014](https://github.com/clojure-emacs/cider/pull/2014): Unify the format for `forms-str` and `arglists-str`.
* [#2027](https://github.com/clojure-emacs/cider/pull/2027): Mark many custom variables as safe
Copy link
Member

Choose a reason for hiding this comment

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

This sentence should end with a .. And the wording can be more specific - "Make all customs variables related to lein and boot params safe.".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@bbatsov bbatsov merged commit 4a7b5ff into clojure-emacs:master Jul 5, 2017
@arichiardi
Copy link
Contributor

Glad it's in!

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.

3 participants