search

PostgREST / postgrest

REST API for any Postgres database
https://postgrest.org
MIT License
23.48k stars 1.03k forks source link

Backport to v12 #3646

Closed wolfgangwalther closed 4 months ago

wolfgangwalther commented 4 months ago

Backports the fixes from #3639 and #3644 to v12.

I think we should make it a policy to backport fix commits immediately after merging them to main. We don't need to release on the backbranch immediately, but we should at least open the PR with the backport. This is because at this time, the changes are much more recent and easier to deal with in case they fail tests or so on the backbranch.

steve-chavez commented 4 months ago

I think we should make it a policy to backport fix commits immediately after merging them to main

Fully agree with that.

steve-chavez commented 4 months ago

Any reason why shouldn't we release a new patch version now? The fixes and the perf increase are really good.

wolfgangwalther commented 4 months ago

Any reason why shouldn't we release a new patch version now? The fixes and the perf increase are really good.

Patch release? Sure, go ahead.

The performance commit is not a fix, though, and thus not backpatched. It's only part of main. I sure don't want to release a major now, we might have more breaking changes to consider before doing that.