typelevel / typelevel/doobie

Add a `values` combinator

Open
#902 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Scala
Stars
2.2k
Forks
379
Avg merge
14m
Merged PRs (30d)
8

Description

As discussed on gitter (https://gitter.im/tpolecat/doobie?at=5cdcafe9252dbb751563b1c7 and below), we should add a values function to map types to (?, ?, [...]) VALUES placeholders.

Referenced code in gitter :

// pre-stacksafe implementation of fragments
def values[A](a: A)(implicit ev: Write[A]): Fragment =
  Fragment(List.fill(ev.length)("?").mkString("(", ", ", ")"), a)

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the pre-stacksafe Fragment implementation and the linked Gitter discussion, then inspect the current Fragment and Write APIs. Done means a values function maps a type to a Fragment containing the corresponding VALUES placeholders.

Written by the indexing model from the issue text.

Assessment

Tech stack
scala
Domain
database
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.