From 23053308b93db5a7f6a0c209dea8b68aaf31ede4 Mon Sep 17 00:00:00 2001
From: Bobbie Soedirgo <bobbie@soedirgo.dev>
Date: Wed, 1 Jun 2022 02:22:05 +0800
Subject: [PATCH] perf: nullsFirst = true by default

BREAKING CHANGE: nullsFirst = true by default

We previously set `nullsFirst = false` by default, but it caused some
perf issues: https://github.com/supabase/postgrest-js/issues/239
---
 src/PostgrestTransformBuilder.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/PostgrestTransformBuilder.ts b/src/PostgrestTransformBuilder.ts
index f0981778..484ab7c2 100644
--- a/src/PostgrestTransformBuilder.ts
+++ b/src/PostgrestTransformBuilder.ts
@@ -46,7 +46,7 @@ export default class PostgrestTransformBuilder<T> extends PostgrestBuilder<T> {
     column: keyof T,
     {
       ascending = true,
-      nullsFirst = false,
+      nullsFirst = true,
       foreignTable,
     }: { ascending?: boolean; nullsFirst?: boolean; foreignTable?: string } = {}
   ): this {