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.

Sign up for GitHub

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

Jump to bottom

Feature/deploy #25

Merged
merged 14 commits into from
Aug 14, 2024
Merged

Feature/deploy #25

merged 14 commits into from
Aug 14, 2024

Conversation

zacdezgeo
Copy link
Collaborator

@zacdezgeo zacdezgeo commented Aug 7, 2024 •
edited
Loading

What I Changed

  • Add Lambda deployment on AWS of API using CDK
  • Change packaging structure of API for deployment
  • Update path references for tests and CI
  • Extract deployment and application settings

How to Test it

  • Check out the deployed docs: space2stats.ds.io/docs!
  • Run through the example notebook with the space2stats.ds.io base url
  • Run the unit tests: python -m pytest

Other Notes

@zacdezgeo zacdezgeo added the enhancement New feature or request label Aug 7, 2024
@zacdezgeo zacdezgeo self-assigned this Aug 7, 2024
@zacdezgeo zacdezgeo requested a review from alukach August 7, 2024 18:19
alukach
alukach approved these changes Aug 12, 2024
View reviewed changes
.gitignore Outdated Show resolved Hide resolved
@zacdezgeo zacdezgeo merged commit 6fdcb39 into main Aug 14, 2024
2 checks passed
@zacdezgeo zacdezgeo deleted the feature/deploy branch August 14, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alukach alukach alukach approved these changes

@bitner bitner Awaiting requested review from bitner

@vincentsarago vincentsarago Awaiting requested review from vincentsarago

Assignees

@zacdezgeo zacdezgeo

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add constraint to API call
2 participants
@zacdezgeo @alukach