Skip to content
This repository was archived by the owner on Oct 9, 2025. It is now read-only.

Conversation

mqp
Copy link
Contributor

@mqp mqp commented Jan 17, 2023

I noticed that my Typescript was complaining because code in PostgrestFilterBuilder uses Object.entries, which is an ES2017 method, but Typescript was only targeting ES2015.

AFAIK ES2017 is supported by all modern browsers, so I suggest just bumping the target. You can see here for some compatibility tables: https://kangax.github.io/compat-table/es2016plus/

If you really like targeting ES2015 for a reason I don't know, you could leave the target alone and use the lib property to express that you expect ES2017 library functions to exist: https://www.typescriptlang.org/tsconfig#lib

Copy link
Member

@soedirgo soedirgo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, ES2017 sounds reasonable. Thanks!

@soedirgo soedirgo merged commit 8f49667 into supabase:master Jan 18, 2023
@github-actions
Copy link

🎉 This PR is included in version 1.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants