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 pypi package publishing in CI/CD #184

Closed
wants to merge 36 commits into from

Conversation

dudoslav
Copy link
Collaborator

@dudoslav dudoslav commented Jan 4, 2024

This PR adds automatic pypi package publishing in CI/CD.

I still need to:

  1. When to execute export
  2. Set export repo to pypi not test.pypi

But first, I need to know if this approach is OK?

@dudoslav dudoslav requested a review from ihnorton January 4, 2024 18:53
Copy link

This pull request has been linked to Shortcut Story #38333: [vs] Set up automated release wheel uploads.

@dudoslav
Copy link
Collaborator Author

dudoslav commented Jan 5, 2024

It looks like the python tests do not pass because my fork repo is missing REST_API_TOKEN or something. Is this ok or should I push this branch to upstream repo and create a new PR?

@dudoslav
Copy link
Collaborator Author

dudoslav commented Jan 5, 2024

https://test.pypi.org/project/tiledb-vector-search/ is the published version on test.pypi under my credentials

@dudoslav
Copy link
Collaborator Author

dudoslav commented Jan 9, 2024

The pipeline now finally passes with a small check:
https://github.com/dudoslav/TileDB-Vector-Search/actions/runs/7459145166

@dudoslav dudoslav self-assigned this Jan 16, 2024
@dudoslav
Copy link
Collaborator Author

I will divide this PR into two smaller ones

@dudoslav dudoslav closed this Jan 22, 2024
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.

1 participant