Conversation
Using my new cookiecutter template: https://github.com/bsoyka/cookiecutters/tree/main/python-library
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
There was a problem hiding this comment.
Pull Request Overview
This PR represents a complete rewrite of the Gravify library to add profile support alongside avatar functionality. The refactor modernizes the codebase with improved architecture, type safety, and expanded capabilities.
- Complete rewrite from a single-class design to a modular package structure with dedicated avatar and profile modules
- Addition of Gravatar profile API integration with comprehensive data models
- Migration from MD5 to SHA256 hashing and replacement of legacy dependencies with modern alternatives
Reviewed Changes
Copilot reviewed 42 out of 45 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/gravify/init.py | Main package initialization exposing AvatarGenerator and ProfileAPI classes |
| src/gravify/profiles/api.py | Profile API client for fetching Gravatar user profiles |
| src/gravify/profiles/models.py | Comprehensive Pydantic data models for profile information |
| src/gravify/avatars/generator.py | Modern avatar URL generator with enum-based options |
| src/gravify/utils.py | Utility functions including SHA256 email hashing |
| src/gravify/rest_adapter.py | HTTP client adapter for API interactions |
| tests/ | Complete test suite rewrite with proper structure and API testing |
| pyproject.toml | Updated build system, dependencies, and tooling configuration |
No description provided.