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

chore: speed up CI #178

Merged
merged 1 commit into from
Mar 13, 2025
Merged

chore: speed up CI #178

merged 1 commit into from
Mar 13, 2025

Conversation

steve-chavez
Copy link
Member

@steve-chavez steve-chavez commented Mar 13, 2025

Before each pg test took almost ~20 mins. See
https://github.com/supabase/pg_net/actions/runs/13710113927/job/38344639067

Now each one takes 2 mins max. This is thanks to cachix and the centralization of the pg tooling on https://github.com/steve-chavez/nxpg. The location of this repo can be changed later.

This does change the way of testing from:

net-with-pg-13 python -m pytest -vv

To:

nxpg -v 13 test

But the speed up is worth it.

@steve-chavez steve-chavez force-pushed the nxpg branch 5 times, most recently from ad7b68f to 8572303 Compare March 13, 2025 20:48
@steve-chavez
Copy link
Member Author

macos-15 fails with weird error:

No matching processes belonging to you were found

https://github.com/supabase/pg_net/actions/runs/13844363306/job/38739273900?pr=178

Will comment for now to unblock this

Before each pg test took almost ~20 mins. See
https://github.com/supabase/pg_net/actions/runs/13710113927/job/38344639067

Now each one takes 2 mins max. This is thanks to `cachix`.

Also a lot of files are deleted since the pg tooling is now centralized
on https://github.com/steve-chavez/nxpg.

This does change the way of testing from:

```
net-with-pg-13 python -m pytest -vv
```

To:

```
nxpg -v 13 test
```

But the speed up is worth it.
@steve-chavez steve-chavez marked this pull request as ready for review March 13, 2025 21:18
@steve-chavez steve-chavez merged commit 519a626 into supabase:master Mar 13, 2025
8 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.

2 participants