-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
Tips for building and contributing to the project #525
Comments
hey @colespencer1453, thanks for your interest in contributing to this library. Highly appreciated! we are using pnpm v8 still, although despite the updated lockfile I dont think there will be any problems when using v9 instead. The test require the dependencies to be built. I just added the missing task definition to turbo for this (2349efd). if you run the test without turbo, you need to run the build manually. I usually just run Let me know if you encounter any further issues or if something else is unclear. I would also appreciate a contributors guide! |
@psteinroe I think I'm making progress, but still struggling to run the tests specifically for postgrest-react-query. If I cd to the folder for it and run pnpm test all of the tests fail. If I run pnpm test from the root then it runs tests for @supabase-cache-helpers/storage-core:test: and all of them are passing. I'm not familiar with turbo so I'm probably missing something here. I have the supabase db up and running without issue. UPDATE: The specific error that I'm getting when running these tests is "The schema must be one of the following: ". Currently debugging. |
@psteinroe looks like the main issue here was the supabase config.toml file had an empty array for schemas exposed in the api. Is this intentional?
|
it is, public and storage are always exposed. What errors do you get? |
that is indeed weird! Are you using the supabase cli version declared in the repo? |
All I see in the repo is "supabase":"latest" in the package.json. I upgraded my cli to latest and I"m still getting the same thing. supabase cli version: |
Looks like I just had a bunch of older supabase docker images cached that needed to get cleared. All is well now. Also wasn't using pnpm supabase so was using latest cli version. Version in the project appears to be 1.183.5. |
Hello!
I just started using this package and I'm wanting to dive deeper into how this is working and potentially contribute after learning more. Do you have any advice for building, running, and testing the project? I downloaded it and ran pnpm i with pnpm 9 and that went rather smoothly. Then I started the supabase instance and ran the tests in the projects. There were lots of failing tests so I wasn't sure if that was expected or I was still missing configuration.
I'm specifically getting this error when running tests:
Error: Failed to resolve entry for package "@supabase-cache-helpers/postgrest-core". The package may have incorrect main/module/exports specified in its package.json.
The text was updated successfully, but these errors were encountered: