Skip to content

Commit e509151

Browse files
authored
Add some slack to allocation tests (#1884)
Motivation: Occasionally one or two allocations are added which we want to ignore in benchmarks where we care about changes in the 1000s. Modifications: - Add some slack to benchmarks - Pass in the config to each of the benchmarks, which was previously dropped on the floor - Add the benchmarks to the formatter Result: Allocation limits allow some slack
1 parent 7af20e2 commit e509151

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

scripts/format.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,7 @@ if "$lint"; then
9494
"${REPO}/Sources" \
9595
"${REPO}/Tests" \
9696
"${REPO}/Plugins" \
97+
"${REPO}/Performance/Benchmarks/Benchmarks/GRPCSwiftBenchmark" \
9798
&& SWIFT_FORMAT_RC=$? || SWIFT_FORMAT_RC=$?
9899

99100
if [[ "${SWIFT_FORMAT_RC}" -ne 0 ]]; then
@@ -113,6 +114,7 @@ elif "$format"; then
113114
"${REPO}/Sources" \
114115
"${REPO}/Tests" \
115116
"${REPO}/Plugins" \
117+
"${REPO}/Performance/Benchmarks/Benchmarks/GRPCSwiftBenchmark" \
116118
&& SWIFT_FORMAT_RC=$? || SWIFT_FORMAT_RC=$?
117119

118120
if [[ "${SWIFT_FORMAT_RC}" -ne 0 ]]; then

0 commit comments

Comments
 (0)