Skip to content

Conversation

@spompea-cfa
Copy link
Contributor

@spompea-cfa spompea-cfa commented Sep 24, 2025

  • Add utilities for working with S3, purging the db, etc.
  • Ensure consumer can work with LocalStack's SQS
  • Add LOG_LEVEL environment variable
  • Add sections to README
  • Other tidying, etc.

@spompea-cfa spompea-cfa changed the title Add utils feat: Add utilities for working with LocalStack & other tidying Sep 25, 2025
@jamesiarmes jamesiarmes requested a review from Copilot September 26, 2025 15:38
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds utilities for working with LocalStack and performs various maintenance tasks. The main purpose is to enable development and testing with LocalStack by providing S3 utilities, configuring LocalStack support in middleware services, and improving logging configuration.

Key changes include:

  • Removed outdated test utilities and sample data files
  • Added new development scripts for S3 operations and database management
  • Enhanced middleware to support LocalStack endpoints and improved logging

Reviewed Changes

Copilot reviewed 14 out of 15 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
middleware/testutils.py Removed entire file containing old test utilities
middleware/sample-data/ Removed sample data files (customers.jsonl, hemingway.txt)
middleware/loglib.py Added LOG_LEVEL environment variable support
middleware/consumer.py Added LocalStack endpoint URL support and reduced log verbosity
middleware/exporter.py Removed ad-hoc test functions
dev-scripts/ Added new utilities for S3 operations and database purging
docker-compose.yaml Added LocalStack configuration for tools container
README.md Major restructuring with LocalStack setup instructions
Dockerfiles Package installation updates for tools and middleware

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jamesiarmes jamesiarmes merged commit 5a2fdfa into main Sep 26, 2025
9 checks passed
@jamesiarmes jamesiarmes deleted the add-utils branch September 26, 2025 15:56
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.

3 participants