/clj-stacktrace

Replace @ in README with deref #28

Merged
merged 1 commit into from Oct 23, 2013
+1 −1

Labels

None yet

Milestone

No milestone

Assignees

No one assigned

2 participants

@voidlily @technomancy
@voidlily
voidlily commented Oct 22, 2013

Some lein plugins, such as lein-ancient, read profiles.clj as EDN. The @ reader macro is not valid EDN, so I changed the example in the README to use deref instead.

Exact usage: https://github.com/xsc/lein-ancient/blob/master/src/leiningen/ancient/utils/io.clj#L93

Reed Lipman Replace @ in README with deref 
Some lein plugins, such as [lein-ancient](https://github.com/xsc/lein-ancient), read profiles.clj as EDN. The @ reader macro is not valid EDN, so I changed the example in the README to use deref instead.

Exact usage: https://github.com/xsc/lein-ancient/blob/master/src/leiningen/ancient/utils/io.clj#L93
 f286703
@technomancy
Collaborator
technomancy commented Oct 23, 2013

I can apply this, but it should be noted that reading profiles.clj as EDN is wrong and should be fixed.

@technomancy technomancy merged commit a47a2c9 into mmcgrana:master Oct 23, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment