The Wayback Machine - https://web.archive.org/web/20201017213009/https://github.com/clojure/clojure/pull/71
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

Define the arguments of reduce’s callback function #71

Open
wants to merge 1 commit into
base: master
from

Conversation

@robatron
Copy link

@robatron robatron commented Apr 3, 2017

Hello, I'm a first-time user of clojure, and thought the reduce core function docs could use clarification regarding its callback function's arguments.

I just noticed this project does not accept pull requests via github, but I figured this PR would be a good way to communicate my intended contribution before learning how to contribute a patch via JIRA.

@@ -6684,15 +6684,15 @@
[x] (instance? java.util.UUID x))

(defn reduce
"f should be a function of 2 arguments. If val is not supplied,

This comment has been minimized.

@robatron

robatron Apr 3, 2017
Author

Apologies for the hard wrap diff gore. I changed the first sentence from:

f should be a function of 2 arguments.

to:

f should be a function of 2 arguments: 1st, the accumulator, and 2nd the current element.

@AeroNotix
Copy link

@AeroNotix AeroNotix commented Apr 3, 2017

You'll get the usual "We don't accept pull requests" comment. You need to use JIRA and file Article 239823 and fill in relevant release forms from Paragrah 63a Alpha, subparagraph 12 and ensure you click "accept trans dimensional JIRA brain uplink" when prompted.

@jafingerhut
Copy link
Contributor

@jafingerhut jafingerhut commented Apr 3, 2017

To be a little more precise :-)

You can find links for "Sign up for a JIRA account" and "Creating Tickets" on this page: http://dev.clojure.org/display/community/Contributing

Instructions for signing a Clojure Contributor Agreement are here, only necessary if you wish to contribute code to Clojure: https://clojure.org/community/contributing

@robatron
Copy link
Author

@robatron robatron commented Apr 3, 2017

@AeroNotix @jafingerhut indeed, hence:

I just noticed this project does not accept pull requests via github, but I figured this PR would be a good way to communicate my intended contribution before learning how to contribute a patch via JIRA.

I.e., is the proposed change acceptable? As a complete clojure newb, I just want to make sure I'm not overlooking something obvious before contributing.

@AeroNotix
Copy link

@AeroNotix AeroNotix commented Apr 3, 2017

^the above is exactly why a complicated contribution process is not helpful to beginners and drive-by pull requests.

@michaelklishin
Copy link

@michaelklishin michaelklishin commented Apr 3, 2017

@robatron all it means that Clojure does not use GitHub pull requests. You will go through a hand crafted 16th century style software development process before you know if your change is worth accepting. They used to ask for a snail mail sent to North Carolina before you could contribute anything (this is not a joke).

@jafingerhut
Copy link
Contributor

@jafingerhut jafingerhut commented Apr 4, 2017

@robatron There is only one person who can give the final answer on whether a contribution is acceptable or not, and that is Rich Hickey. In practice, Alex Miller has a pretty good feel for what is interesting or not, and he reads the Clojure Google group where anyone can publish messages, and he often answers questions like this. Or, create a JIRA ticket (no signing of a Clojure CA required), and he can address that question.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.
Morty Proxy This is a proxified and sanitized view of the page, visit original site.