Merge pull request #2379 from typelevel/minor_build_fixes

Minor build fixes

Author: jatcwang

.github/workflows/ci.yml

@@ -10,7 +10,6 @@ name: Continuous Integration
 on:
   pull_request:
     branches: ['**', '!update/**', '!pr/**']
-    types: [locked]
   push:
     branches: ['**', '!update/**', '!pr/**']
     tags: [v*]

build.sbt

@@ -64,10 +64,6 @@ ThisBuild / githubWorkflowBuild := {
     case other => other
   }
 }
-// Disable running CI for Pull Requests (a normal push already triggers CI)
-// We set an uncommon status instead of empty because passing an empty list seems to make
-// GHA use the default trigger event types..
-ThisBuild / githubWorkflowPREventTypes := Seq(PREventType.Locked)
 ThisBuild / githubWorkflowBuildPostamble ++= Seq(
   WorkflowStep.Sbt(
     commands = List("checkGitNoUncommittedChanges"),

docker-compose.yml

@@ -1,5 +1,3 @@
-version: "3.1"
-
 services:
   postgres:
     image: postgis/postgis:16-3.4

modules/core/src/main/scala/doobie/hi/resultset.scala

@@ -175,7 +175,7 @@ object resultset {
     }
 
   /** Equivalent to `getNext`, but verifies that there is exactly one row remaining.
-    * @throws UnexpectedCursorPosition
+    * @throws doobie.util.invariant.UnexpectedCursorPosition
     *   if there is not exactly one row remaining
     * @group Results
     */
@@ -187,7 +187,7 @@ object resultset {
     }
 
   /** Equivalent to `getNext`, but verifies that there is at most one row remaining.
-    * @throws UnexpectedContinuation
+    * @throws doobie.util.invariant.UnexpectedContinuation
     *   if there is more than one row remaining
     * @group Results
     */
@@ -199,7 +199,7 @@ object resultset {
     }
 
   /** Consumes the remainder of the resultset, but verifies that there is at least one row remaining.
-    * @throws UnexpectedEnd
+    * @throws doobie.util.invariant.UnexpectedEnd
     *   if there is not at least one row remaining
     * @group Results
     */

modules/core/src/main/scala/doobie/package.scala

@@ -33,7 +33,7 @@ package object doobie
       *   1. You're NOT using one of the database doobie has direct java.time isntances for (PostgreSQL / MySQL). (They
       *      have more accurate column type checks) 2. Your driver natively supports java.time.* types
       *
-      * If your driver doesn't support java.time.* types, use [[doobie.implicits.legacy.instant/localdate]] instead
+      * If your driver doesn't support java.time.* types, use `doobie.implicits.legacy.instant/localdate` instead
       */
     object javatimedrivernative extends TimeMetaInstances
   }

modules/core/src/main/scala/doobie/util/query.scala

@@ -40,8 +40,8 @@ object query {
       */
     def sql: String
 
-    /** An optional [[Pos]] indicating the source location where this [[Query]] was constructed. This is used only for
-      * diagnostic purposes.
+    /** An optional [[doobie.util.pos.Pos]] indicating the source location where this [[Query]] was constructed. This is
+      * used only for diagnostic purposes.
       * @group Diagnostics
       */
     def pos: Option[Pos]

modules/core/src/main/scala/doobie/util/testing.scala

@@ -59,7 +59,7 @@ package testing {
     val header: String = show"$typeName defined at $location"
   }
 
-  /** Information from [[Analysis]], prepared for output. */
+  /** Information from [[doobie.util.analysis.Analysis]], prepared for output. */
   final case class AnalysisReport(
       header: String,
       sql: Block,

modules/core/src/main/scala/doobie/util/transactor.scala

@@ -109,7 +109,7 @@ object transactor {
     /** A `Strategy` for running a program on a connection * */
     def strategy: Strategy
 
-    /** Construct a [[Yolo]] for REPL experimentation. */
+    /** Construct a [[doobie.util.yolo.Yolo]] for REPL experimentation. */
     def yolo(implicit ev: Async[M]): Yolo[M] = new Yolo(this)
 
     /** Construct a program to perform arbitrary configuration on the kernel value (changing the timeout on a connection

modules/core/src/main/scala/doobie/util/update.scala

@@ -46,8 +46,8 @@ object update {
       */
     val sql: String
 
-    /** An optional [[Pos]] indicating the source location where this [[Update]] was constructed. This is used only for
-      * diagnostic purposes.
+    /** An optional [[doobie.util.pos.Pos]] indicating the source location where this [[Update]] was constructed. This
+      * is used only for diagnostic purposes.
       * @group Diagnostics
       */
     val pos: Option[Pos]
@@ -297,8 +297,8 @@ object update {
       */
     val sql: String
 
-    /** An optional [[Pos]] indicating the source location where this [[Update0]] was constructed. This is used only for
-      * diagnostic purposes.
+    /** An optional [[doobie.util.pos.Pos]] indicating the source location where this [[Update0]] was constructed. This
+      * is used only for diagnostic purposes.
       * @group Diagnostics
       */
     val pos: Option[Pos]

modules/free/src/main/scala/doobie/WeakAsync.scala

@@ -51,7 +51,7 @@ object WeakAsync {
   def liftK[F[_], G[_]](implicit F: Async[F], G: WeakAsync[G]): Resource[F, F ~> G] =
     Dispatcher.parallel[F].map(new Lifter(_))
 
-  /** Like [[liftK]] but specifically returns a [[LiftIO]] */
+  /** Like [[liftK]] but specifically returns a [[cats.effect.LiftIO]] */
   def liftIO[F[_]](implicit F: WeakAsync[F]): Resource[IO, LiftIO[F]] =
     Dispatcher.parallel[IO].map(new Lifter(_) with LiftIO[F] {
       def liftIO[A](ioa: IO[A]) = super[Lifter].apply(ioa)