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

Feature/ci cd #556

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

eligrinfeld
Copy link

No description provided.

- Add utility functions for business ID generation
- Improve database service with proper types
- Fix type safety issues in search implementation
- Add caching layer for search results
- Add database initialization scripts
- Add configuration files
- Add documentation
- Add public assets
- Add source code structure
- Update README
- Added confidence scoring system for data quality
- Implemented strict validation for emails, phones, and addresses
- Added batch processing to prevent LLM overload
- Improved error handling and fallback mechanisms
- Added caching based on confidence scores

Technical changes:
- Added regex validation for contact info
- Implemented scoring system (0-1 scale)
- Added timeout protection for LLM calls
- Enhanced post-processing for consistent formatting
- Added business type detection for context

Breaking changes: None
Dependencies: No new dependencies required
- Added confidence scoring system (0-1) for data quality
- Implemented strict validation for contact info
- Added batch processing and timeout protection
- Improved error handling with fallbacks
- Added smart caching based on confidence scores

Technical changes:
- Added regex validation for emails, phones, addresses
- Implemented business type detection
- Enhanced post-processing for consistent formatting
- Added JSDoc comments for maintainability

Testing:
- Verified with restaurant and plumber searches
- Confirmed improved data quality
- Validated timeout handling
@eligrinfeld
Copy link
Author

Add continuous integration and deployment workflow that:

  • Runs automated tests
  • Checks code formatting
  • Generates test coverage reports
  • Builds the application
  • Deploys to staging/production environments

Added:

  • GitHub Actions workflow configuration
  • Test environment setup
  • Coverage reporting with Codecov
  • Deployment configuration

- Separate server setup from initialization\n- Add port availability check utility\n- Fix double server start issue\n- Improve error handling for port conflicts
- Add business routes and middleware\n- Update search and database services\n- Improve health check implementation\n- Update CI workflow configuration
@ItzCrazyKns
Copy link
Owner

Hey, can I know what this PR is for? The description isn't quite explanatory.

@eligrinfeld
Copy link
Author

What do you mean?

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