Skip to content

Supabase py#302714

Closed
samrose wants to merge 3 commits intoNixOS:masterfrom
samrose:supabase-py
Closed

Supabase py#302714
samrose wants to merge 3 commits intoNixOS:masterfrom
samrose:supabase-py

Conversation

@samrose
Copy link
Contributor

@samrose samrose commented Apr 9, 2024 •
edited
Loading

Description of changes

added supabase-py python client for supabase and it's dependencies

https://github.com/supabase-community/supabase-py

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@github-actions github-actions bot added the 6.topic: python Python is a high-level, general-purpose programming language. label Apr 9, 2024
@samrose samrose self-assigned this Apr 9, 2024
@ofborg ofborg bot added 8.has: package (new) This PR adds a new package 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Apr 9, 2024
risicle
risicle reviewed Apr 9, 2024
View reviewed changes
pkgs/development/python-modules/supabase_auth/default.nix
}:

python3Packages.buildPythonPackage rec {
pname = "gotrue";
Copy link
Contributor

@risicle risicle Apr 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

Copy link
Contributor Author

@samrose samrose Apr 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@risicle it's had a recent name change, but I guess we'd want to use the name as found pypi, so I'll revise. Thanks!

@risicle
Copy link
Contributor

risicle commented Apr 9, 2024

This should be one commit per added package, with the commit message as specified in CONTRIBUTING.md.

If the tests aren't in the pypi packages, have you tried pulling the source straight from github? Otherwise, what with these packages also not having pythonImportsCheck, they will get broken without anyone noticing.

@samrose
Copy link
Contributor Author

samrose commented Apr 10, 2024

This should be one commit per added package, with the commit message as specified in CONTRIBUTING.md.

If the tests aren't in the pypi packages, have you tried pulling the source straight from github? Otherwise, what with these packages also not having pythonImportsCheck, they will get broken without anyone noticing.

Thanks, I'll revise that per your feedback.

@samrose samrose closed this by deleting the head repository Nov 3, 2024
@dddictionary
Copy link

dddictionary commented Apr 6, 2025

Hi, I am in need of this package to be added into nixpkgs. Is there anything I can do to help push this?

@Siegema
Copy link
Contributor

Siegema commented Apr 15, 2025

@dddictionary
I'm trying to get this cleared because I needed it as well
#398562

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@risicle risicle risicle left review comments

Assignees

@samrose samrose

Labels

6.topic: python Python is a high-level, general-purpose programming language. 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@samrose @risicle @dddictionary @Siegema