Skip to content

Feature: Improve local development setup#354

Merged
camfairchild merged 7 commits intomainfrom
feat/improve_docs_local_setup
Apr 30, 2026
Merged

Feature: Improve local development setup#354
camfairchild merged 7 commits intomainfrom
feat/improve_docs_local_setup

Conversation

@jmnmv12
Copy link
Copy Markdown
Contributor

@jmnmv12 jmnmv12 commented Apr 29, 2026

Changelog

This PR introduces a couple of changes to improve the local development setup:

  • Add a new optional config S3_endpoint_url that allows the user to specify a different S3 URL from the standard AWS one. This is especially useful when using S3 mocks locally
  • Small refactor on the utils/s3.py logic to aggregate the s3 client initialization logic into a single method
  • Add a docker-compose.yaml file that allows the developer to run the Postgres DB, Adobe S3 mock and the api service with a single command. This should only be used for development purposes
  • Improve root project documentation on how to run the services locally

@jmnmv12 jmnmv12 changed the title Feature: Improve local development setup [WIP] Feature: Improve local development setup Apr 29, 2026
@jmnmv12 jmnmv12 marked this pull request as ready for review April 29, 2026 14:21
@jmnmv12 jmnmv12 changed the title [WIP] Feature: Improve local development setup Feature: Improve local development setup Apr 29, 2026
Co-authored-by: Copilot <copilot@github.com>
@clementblaise
Copy link
Copy Markdown
Collaborator

clementblaise commented Apr 30, 2026

In my branch for openrouter I had made the changes for S3 endpoint, should I removed them or cherry picking here makes more sense?

@jmnmv12
Copy link
Copy Markdown
Contributor Author

jmnmv12 commented Apr 30, 2026

In my branch for openrouter I had made the changes for S3 endpoint, should I removed them or cherry picking here makes more sense?

Me and @clementblaise synced and since this PR is smaller and will probably be merged first we think it's best to keep the S3 changes here and then Clement can merge these changes into his branch 👌

@camfairchild camfairchild merged commit dafb585 into main Apr 30, 2026
2 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.

4 participants