Add upper bounds and fix hlint issue

Change-Id: Ie6d1e8827872b53127528a19e83441ef19b457d0

Author: TristanCacqueray

.zuul.yaml

@@ -1,17 +1,18 @@
 - project:
     check: &jobs
       jobs:
-        - cabal-test:
+        - run-test-command:
             vars:
-              ghc_version: 8.8
-              cabal_install_args: "--lib"
+              test_command:
+                - cabal build -O0 --enable-tests
+                - cabal test  -O0 --test-show-details=direct
             nodeset:
               nodes:
                 - name: container
-                  label: zuul-worker-haskell
+                  label: zuul-worker-haskell-ghc9
         - hlint:
             nodeset:
               nodes:
                 - name: container
-                  label: zuul-worker-haskell
+                  label: zuul-worker-haskell-ghc9
     gate: *jobs

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.6.1
+
+- Apply hlint suggestion
+- Add version bounds to the build-depends
+
 ## 0.1.6.0
 
 - Add a nix flake

gerrit.cabal

@@ -1,6 +1,6 @@
 cabal-version:       2.4
 name:                gerrit
-version:             0.1.6.0
+version:             0.1.6.1
 synopsis:            A gerrit client library
 description:
     Gerrit is a client library to interface with https://www.gerritcodereview.com/
@@ -26,10 +26,10 @@ source-repository head
   location:            https://github.com/softwarefactory-project/gerrit-haskell.git
 
 common common-options
-  build-depends:       base                   >= 4.11.0.0
-                     , aeson                  >= 1.0.0.0
-                     , bytestring
-                     , time                   >= 1.9.3
+  build-depends:       base                   >= 4.11 && < 5
+                     , aeson                  >= 2.0  && < 2.3
+                     , bytestring             >= 0.10 && < 0.13
+                     , time                   >= 1    && < 2
   ghc-options:         -Wall
                        -Wcompat
                        -Widentities
@@ -48,11 +48,11 @@ common common-options
 
 library
   import:              common-options
-  build-depends:       containers             >= 0.5.8.0
-                     , http-client            >= 0.5.0
-                     , http-client-openssl    >= 0.3.0
-                     , text                   >= 0.11.1.0
-                     , aeson-casing           >= 0.2.0.0
+  build-depends:       containers             >= 0.5.8.0 && < 0.8
+                     , http-client            >= 0.5.0   && < 0.8
+                     , http-client-openssl    >= 0.3.0   && < 1
+                     , text                   >= 0.11    && < 3
+                     , aeson-casing           >= 0.2.0.0 && < 0.4
 
   hs-source-dirs:      src
   exposed-modules:     Gerrit
@@ -68,10 +68,10 @@ test-suite gerrit-test
   type:                exitcode-stdio-1.0
   hs-source-dirs:      test
   main-is:             Spec.hs
-  build-depends:       directory
+  build-depends:       directory                  >= 1.0 && < 2
                      , gerrit
-                     , tasty                      >= 0.11.2
-                     , tasty-hunit                >= 0.10
+                     , tasty                      >= 0.11.2 && < 2
+                     , tasty-hunit                >= 0.10   && < 2
   ghc-options:         -threaded
                        -rtsopts
                        -with-rtsopts=-N

src/Gerrit/Data/Change.hs

@@ -196,7 +196,7 @@ instance FromJSON GerritTime where
   parseJSON = withText "UTCTimePlus" (parse . T.unpack)
     where
       format = "%F %T.000000000"
-      tryParse f s = parseTimeM False defaultTimeLocale f s
+      tryParse = parseTimeM False defaultTimeLocale
       parse s = GerritTime <$> tryParse format s
 
 data GerritChangeMessage = GerritChangeMessage