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 CI to vectorizing CU-86bvyavpu #46

Merged
merged 16 commits into from
Sep 21, 2023

Conversation

pietrobolcato
Copy link
Collaborator

@pietrobolcato pietrobolcato commented Sep 20, 2023

This PR includes:

  • CI pipeline to run tests with github actions
  • System dependencies installer script shared between dev container and CI, to keep dependencies consistent and to make them easier to update in the future
  • Update README documentation
  • Remove python-dev as system dependencies, as it was not used

CU Task: https://app.clickup.com/t/86bvyavpu

Copy link
Member

@phil-r phil-r left a comment

Choose a reason for hiding this comment

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

lgtm

@pietrobolcato pietrobolcato merged commit 93e4675 into main Sep 21, 2023
2 checks passed
@pietrobolcato pietrobolcato deleted the CU-86bvyavpu-Add-CI-to-vectorizing branch September 21, 2023 07:50
bhvishal9 added a commit that referenced this pull request Aug 23, 2024
* Add CI to vectorizing CU-86bvyavpu  (#46)

* Add CI pipeline to run tests with gha
* Add system dependencies installer script shared between dev container and CI
* Update README documentation
* Remove 'python-dev0 as system dependency

* Upgrade vectorizing to python 3.11 CU-866aw6ung (#42)

* Update dockerfile
* Update requirements
* Update environments

* Add linting and formatting CU-86bw0yqjw  (#47)

* Linting and formatting setup using pre-commit
* Updated devcontainer with pre-commit

* Fix k8s deployment CU-86bwkb51r

* Add PR template CU-86bx2ntyx  (#50)

* Add PR template to the repository, with information aligned with guidelines in notion

* Remove PR template to use org one (#51)

* Remove PR template to use org one

* Fix UA  (#52)

* Update UA

* CU-86bze09vt Move vectorizing to new clusters (#53)

* Update Dockerfile and remove AWS AK/SAK

* Update deployment workflow

* Minor modification to the deploy workflow

* Add region_name to the client

* Upgrade boto3

* Update port

* Undo temp changes

---------

Co-authored-by: Pietro Bolcato <[email protected]>
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