Skip to content

Commit 72659b0

Browse files
committed
Avoid multiple filters
1 parent f732d0b commit 72659b0

File tree

1 file changed

+19
-15
lines changed

1 file changed

+19
-15
lines changed

src/FSharp.Formatting.ApiDocs/Categorise.fs

Lines changed: 19 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -53,28 +53,32 @@ let entities (nsIndex: int, ns: ApiDocNamespace, suppress) =
5353
// TODO: use <exclude /> to do these, or work out if there's a better way
5454
if suppress then
5555
categoryEntities
56-
57-
// Remove FSharp.Data.UnitSystems.SI from the list-of-namespaces
58-
// display - it's just so rarely used, has long names and dominates the docs.
59-
//
60-
// See https://github.com/fsharp/fsharp-core-docs/issues/57, we may rethink this
61-
|> List.filter (fun e ->
62-
(e.Symbol.Namespace <> Some "Microsoft.FSharp.Data.UnitSystems.SI.UnitSymbols"))
63-
|> List.filter (fun e ->
64-
(e.Symbol.Namespace <> Some "Microsoft.FSharp.Data.UnitSystems.SI.UnitNames"))
65-
// Don't show 'AnonymousObject' in list-of-namespaces navigation
6656
|> List.filter (fun e ->
57+
// Remove FSharp.Data.UnitSystems.SI from the list-of-namespaces
58+
// display - it's just so rarely used, has long names and dominates the docs.
59+
//
60+
// See https://github.com/fsharp/fsharp-core-docs/issues/57, we may rethink this
61+
(e.Symbol.Namespace <> Some "Microsoft.FSharp.Data.UnitSystems.SI.UnitSymbols")
62+
&& (e.Symbol.Namespace <> Some "Microsoft.FSharp.Data.UnitSystems.SI.UnitNames")
63+
64+
&&
65+
66+
// Don't show 'AnonymousObject' in list-of-namespaces navigation
6767
not (
6868
e.Symbol.Namespace = Some "Microsoft.FSharp.Linq.RuntimeHelpers"
6969
&& e.Symbol.DisplayName = "AnonymousObject"
70-
))
71-
// Don't show 'FSharp.Linq.QueryRunExtensions' in list-of-namespaces navigation
72-
|> List.filter (fun e ->
70+
)
71+
72+
&&
73+
74+
// Don't show 'FSharp.Linq.QueryRunExtensions' in list-of-namespaces navigation
7375
not (
7476
e.Symbol.Namespace = Some "Microsoft.FSharp.Linq.QueryRunExtensions"
7577
&& e.Symbol.DisplayName = "LowPriority"
76-
))
77-
|> List.filter (fun e ->
78+
)
79+
80+
&&
81+
7882
not (
7983
e.Symbol.Namespace = Some "Microsoft.FSharp.Linq.QueryRunExtensions"
8084
&& e.Symbol.DisplayName = "HighPriority"

0 commit comments

Comments
 (0)