Add OFFSET

[gijsaddas]

Introduces OFFSET argument when using the LIMIT function

[sourcery-ai]

Sourcery Code Quality Report

✅  Merging this PR will increase code quality in the affected files by 0.16%.

Quality metrics	Before	After	Change
Complexity	1.18 ⭐	1.16 ⭐	-0.02 👍
Method Length	35.71 ⭐	35.46 ⭐	-0.25 👍
Working memory	5.90 ⭐	5.87 ⭐	-0.03 👍
Quality	85.74% ⭐	85.90% ⭐	0.16% 👍

Other metrics	Before	After	Change
Lines	360	371	11

Changed files	Quality Before	Quality After	Quality Change
postgrest/base_request_builder.py	85.74% ⭐	85.90% ⭐	0.16% 👍

Here are some functions in these files that still need a tune-up:

File	Function	Complexity	Length	Working Memory	Quality	Recommendation

Legend and Explanation

The emojis denote the absolute quality of the code:

• ⭐ excellent
• 🙂 good
• 😞 poor
• ⛔ very poor

The 👍 and 👎 indicate whether the quality has improved or gotten worse with this pull request.

---

Please see our documentation here for details on how these metrics are calculated.

We are actively working on this report - lots more documentation and extra metrics to come!

Help us improve this quality report!

[J0]

Hey!

Wanted to check in on the PR - is the setup.py file needed? Otherwise it seems fine to me - might want to consider merging in the Sorcery refactoring

[deleted]

Hey, this seems like a pretty useful addition to the API. Just wanted to check in on what the status of merging it. (Looks to me like the setup.py could be removed completely since a pyproject.toml exists).

If you came here just looking for a way to do pagination, be advised that a range() method also exists on the query builder, though beware that it decrements its end parameter before emitting the header for some reason:

postgrest-py/postgrest_py/base_request_builder.py

Line 366 in 29e91a2

def range(self: _FilterT, start: int, end: int) -> _FilterT:

[J0]

@deleted I'll make the changes and merge it if there's no update by end of day UTC. I do sometimes lose track though - so feel free to bump if I don't get back by then.

[J0]

Merged in via #326

thanks for the contribution!