From 357e28e794bf8f2801ac4bbf2c979a8eb8e3b826 Mon Sep 17 00:00:00 2001
From: Nicholas Bauer <nicholasbauer@outlook.com>
Date: Thu, 24 Dec 2020 11:58:31 -0500
Subject: [PATCH 1/3] Reference functions instead of methods

By referencing methods sometimes and not just functions, there was some duplication and confusion. For instance, we had `get(::Any, ::Any, ::Any)` method and also a `get` function, the latter of which duplicated the documentation for the former.

Is there an unseen benefit to referencing just the function at times?

If this PR is acceptable, I can go through the other pages to do the same.
---
 doc/src/base/collections.md | 32 ++++++++++++--------------------
 1 file changed, 12 insertions(+), 20 deletions(-)

diff --git a/doc/src/base/collections.md b/doc/src/base/collections.md
index 347d2d33724d4..3eb62b737346e 100644
--- a/doc/src/base/collections.md
+++ b/doc/src/base/collections.md
@@ -93,9 +93,9 @@ Base.indexin
 Base.unique
 Base.unique!
 Base.allunique
-Base.reduce(::Any, ::Any)
-Base.foldl(::Any, ::Any)
-Base.foldr(::Any, ::Any)
+Base.reduce
+Base.foldl
+Base.foldr
 Base.maximum
 Base.maximum!
 Base.minimum
@@ -111,32 +111,26 @@ Base.sum
 Base.sum!
 Base.prod
 Base.prod!
-Base.any(::Any)
-Base.any(::AbstractArray, ::Any)
+Base.any
 Base.any!
-Base.all(::Any)
-Base.all(::AbstractArray, ::Any)
+Base.all
 Base.all!
 Base.count
-Base.any(::Any, ::Any)
-Base.all(::Any, ::Any)
 Base.foreach
 Base.map
 Base.map!
-Base.mapreduce(::Any, ::Any, ::Any)
-Base.mapfoldl(::Any, ::Any, ::Any)
-Base.mapfoldr(::Any, ::Any, ::Any)
+Base.mapreduce
+Base.mapfoldl
+Base.mapfoldr
 Base.first
 Base.last
 Base.front
 Base.tail
 Base.step
-Base.collect(::Any)
-Base.collect(::Type, ::Any)
+Base.collect
 Base.filter
 Base.filter!
-Base.replace(::Any, ::Pair...)
-Base.replace(::Base.Callable, ::Any)
+Base.replace
 Base.replace!
 Base.rest
 ```
@@ -200,13 +194,11 @@ Base.IdDict
 Base.WeakKeyDict
 Base.ImmutableDict
 Base.haskey
-Base.get(::Any, ::Any, ::Any)
 Base.get
-Base.get!(::Any, ::Any, ::Any)
-Base.get!(::Function, ::Any, ::Any)
+Base.get!
 Base.getkey
 Base.delete!
-Base.pop!(::Any, ::Any, ::Any)
+Base.pop!
 Base.keys
 Base.values
 Base.pairs

From b801d32afcb64402494599417f47597a8b84f22a Mon Sep 17 00:00:00 2001
From: Nicholas Bauer <nicholasbauer@outlook.com>
Date: Mon, 28 Dec 2020 10:14:52 -0500
Subject: [PATCH 2/3] Update collections.md

---
 doc/src/base/collections.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/src/base/collections.md b/doc/src/base/collections.md
index 3eb62b737346e..fc66612824cd6 100644
--- a/doc/src/base/collections.md
+++ b/doc/src/base/collections.md
@@ -198,7 +198,7 @@ Base.get
 Base.get!
 Base.getkey
 Base.delete!
-Base.pop!
+Base.pop!(::Any, ::Any, ::Any)
 Base.keys
 Base.values
 Base.pairs

From 436a9d333830cbefb84b7602f69d44cb237464cf Mon Sep 17 00:00:00 2001
From: Nicholas Bauer <nicholasbauer@outlook.com>
Date: Mon, 22 Feb 2021 16:19:54 -0500
Subject: [PATCH 3/3] Narrowing scope

---
 doc/src/base/collections.md | 26 ++++++++++++++++----------
 1 file changed, 16 insertions(+), 10 deletions(-)

diff --git a/doc/src/base/collections.md b/doc/src/base/collections.md
index fc66612824cd6..fe4d8f80b1cd0 100644
--- a/doc/src/base/collections.md
+++ b/doc/src/base/collections.md
@@ -93,9 +93,9 @@ Base.indexin
 Base.unique
 Base.unique!
 Base.allunique
-Base.reduce
-Base.foldl
-Base.foldr
+Base.reduce(::Any, ::Any)
+Base.foldl(::Any, ::Any)
+Base.foldr(::Any, ::Any)
 Base.maximum
 Base.maximum!
 Base.minimum
@@ -111,26 +111,32 @@ Base.sum
 Base.sum!
 Base.prod
 Base.prod!
-Base.any
+Base.any(::Any)
+Base.any(::AbstractArray, ::Any)
 Base.any!
-Base.all
+Base.all(::Any)
+Base.all(::AbstractArray, ::Any)
 Base.all!
 Base.count
+Base.any(::Any, ::Any)
+Base.all(::Any, ::Any)
 Base.foreach
 Base.map
 Base.map!
-Base.mapreduce
-Base.mapfoldl
-Base.mapfoldr
+Base.mapreduce(::Any, ::Any, ::Any)
+Base.mapfoldl(::Any, ::Any, ::Any)
+Base.mapfoldr(::Any, ::Any, ::Any)
 Base.first
 Base.last
 Base.front
 Base.tail
 Base.step
-Base.collect
+Base.collect(::Any)
+Base.collect(::Type, ::Any)
 Base.filter
 Base.filter!
-Base.replace
+Base.replace(::Any, ::Pair...)
+Base.replace(::Base.Callable, ::Any)
 Base.replace!
 Base.rest
 ```