Skip to content

Fixed buid errors - #52 #53

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

Merged
merged 1 commit into from
Jan 4, 2023
Merged

Conversation

glowcap
Copy link
Contributor

@glowcap glowcap commented Jan 3, 2023

What kind of change does this PR introduce?

Fixes build errors mentioned in issue #52 by:

  • Updating Package file
  • Updating SupabaseClient.swift

What is the current behavior?

Fails to build due to breaking changes in postgrest-swift

What is the new behavior?

Now builds and runs as expected

Additional context

Glad to help!

@grdsdev
Copy link
Collaborator

grdsdev commented Jan 3, 2023

Hy @glowcap thanks for this.

But the #45 will already solve this, I just need a bit more testing for merging the RC.

@glowcap
Copy link
Contributor Author

glowcap commented Jan 3, 2023

Hi @GRSouza. I understand, but your PR goes way beyond the scope of the fix and will probably take more time to review. In the meantime, people like myself can't reset package caches or run any SPM updates without running into this issue.

This simply let's people move forward with their work while you wrap up your RC branch and allow it to be properly reviewed and merged

@grdsdev grdsdev merged commit 1329339 into supabase:master Jan 4, 2023
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.

2 participants