Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add x-region support #126

Merged
merged 12 commits into from
Jul 25, 2024
Merged

Add x-region support #126

merged 12 commits into from
Jul 25, 2024

Conversation

juancarlospaco
Copy link
Contributor

@juancarlospaco juancarlospaco commented Jul 22, 2024

What kind of change does this PR introduce?

  • Add x-region support, which allows for the specification of an edge region.
  • Tiny code optimization/reordering, check for invoke_options only once.

@juancarlospaco juancarlospaco marked this pull request as draft July 22, 2024 20:25
@codecov-commenter
Copy link

codecov-commenter commented Jul 22, 2024

Codecov Report

Attention: Patch coverage is 88.88889% with 2 lines in your changes missing coverage. Please review.

Project coverage is 86.29%. Comparing base (a809e98) to head (0588be9).
Report is 51 commits behind head on main.

Files Patch % Lines
supabase_functions/_async/functions_client.py 88.88% 1 Missing ⚠️
supabase_functions/_sync/functions_client.py 88.88% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #126      +/-   ##
==========================================
+ Coverage   85.21%   86.29%   +1.07%     
==========================================
  Files           7        7              
  Lines         115      124       +9     
==========================================
+ Hits           98      107       +9     
  Misses         17       17              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@juancarlospaco juancarlospaco marked this pull request as ready for review July 24, 2024 23:54
@silentworks silentworks merged commit 59135d8 into supabase:main Jul 25, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants