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

feat: Retry #571

Closed
wants to merge 32 commits into from
Closed

feat: Retry #571

wants to merge 32 commits into from

Conversation

juancarlospaco
Copy link
Contributor

@juancarlospaco juancarlospaco commented Feb 7, 2025 •
edited
Loading

What kind of change does this PR introduce?

  • Retry

DO NOT MERGE

@juancarlospaco juancarlospaco changed the title TBD: WIP feat: TBD: WIP Feb 7, 2025
@juancarlospaco juancarlospaco changed the title feat: TBD: WIP feat: Retry: WIP Feb 11, 2025
@coveralls
Copy link

coveralls commented Feb 14, 2025 •
edited
Loading

Pull Request Test Coverage Report for Build 13321035087

Details

  • 53 of 72 (73.61%) changed or added relevant lines in 8 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall first build on retry at 94.622%
Changes Missing Coverage Covered Lines Changed/Added Lines %
postgrest/_async/client.py 5 6 83.33%
postgrest/_async/request_builder.py 18 26 69.23%
postgrest/_sync/request_builder.py 19 29 65.52%
Totals Coverage Status
Change from base Build 13261920679: 94.6%
Covered Lines: 1777
Relevant Lines: 1878
💛 - Coveralls

@juancarlospaco juancarlospaco changed the title feat: Retry: WIP feat: Retry Feb 14, 2025
@juancarlospaco juancarlospaco marked this pull request as ready for review February 14, 2025 18:13
@grdsdev grdsdev closed this Feb 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@grdsdev grdsdev Awaiting requested review from grdsdev grdsdev is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@juancarlospaco @coveralls @grdsdev