Skip to content

Issues3 - #5980

Closed
AleksandrSherman wants to merge 8 commits into
betafrom
issues3
Closed

Issues3#5980
AleksandrSherman wants to merge 8 commits into
betafrom
issues3

Conversation

@AleksandrSherman

@AleksandrSherman AleksandrSherman commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Tests from issue have been added

Worth noticing:

1.

  • inArray

    if (values.length === 0) {
      return sql`false`;
    }

    was replaced with

    if (values.length === 0) {
      return sql`1 = 0`;
    }
  • notInArray

    if (values.length === 0) {
      return sql`true`;
    }

    was replaced with

    if (values.length === 0) {
      return sql`1 = 1`;
    }

MSSQL does not support TRUE/FALSE literals in this context. Using 1 = 0 and 1 = 1 is the agnostic solution

2. Removed strict flag from not only cli, but from drizzle.config.ts types

@AleksandrSherman

AleksandrSherman commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator Author

Merged into issues4 that will be merged into rc5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants