Skip to content

Commit 880b734

Browse files
committed
Removes extra-deps requirement
Realized that the reason for the extra whitespace was that some of the results were being passed through `pShow` twice which was escaping some characters and adding whitespace. This commit prevents that behavior by replacing one usage of pShow with a composition of show and Text.pack which prevents that from happening.
1 parent d7e4b39 commit 880b734

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

src/Testing/CurlRunnings/Types.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -361,7 +361,7 @@ instance Show CaseResult where
361361
T.unpack $ makeRed "[FAIL] " <>
362362
name c <>
363363
"\n" <>
364-
mconcat (map ((\s -> "\nAssertion failed: " <> s) . (<> "\n") . pShow) failures)
364+
mconcat (map ((\s -> "\nAssertion failed: " <> s) . (<> "\n") . (T.pack . show)) failures)
365365

366366
-- | A wrapper type around a set of test cases. This is the top level spec type
367367
-- that we parse a test spec file into

stack.yaml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,7 @@ packages:
4040
# Dependency packages to be pulled from upstream that are not in the resolver
4141
# (e.g., acme-missiles-0.3)
4242
# extra-deps: []
43-
extra-deps:
44-
- pretty-simple-2.1.0.1
43+
extra-deps: []
4544

4645
# Override default flag values for local packages and extra-deps
4746
# flags: {}

0 commit comments

Comments
 (0)