Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Moohan pushed 1 commit to env_vars Moohan/gartan_scraper_bot
- Update app.py Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> 92bd354
Moohan pushed 1 commit to env_vars Moohan/gartan_scraper_bot
- Update app.py Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> 372fb95
Moohan pushed 1 commit to env_vars Moohan/gartan_scraper_bot
- Update Dockerfile Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> 5f4d542
coderabbitai[bot] created a review comment on a pull request on Moohan/gartan_scraper_bot
_:hammer_and_wrench: Refactor suggestion_ **Enhance login error handling** Whilst the status code check is good, consider handling connection errors and timeouts. Also, different status codes mig...
coderabbitai[bot] created a review comment on a pull request on Moohan/gartan_scraper_bot
_:hammer_and_wrench: Refactor suggestion_ **Enhance error reporting for missing environment variables** The current implementation doesn't specify which environment variable is missing. Consider ...
coderabbitai[bot] created a review comment on a pull request on Moohan/gartan_scraper_bot
_:hammer_and_wrench: Refactor suggestion_ **Improve data retrieval error handling** Similar to the login process, consider enhancing the error handling for data retrieval with timeout and connect...
coderabbitai[bot] created a review comment on a pull request on Moohan/gartan_scraper_bot
_:hammer_and_wrench: Refactor suggestion_ **Consider using os.environ.get() with default values** Whilst `os.getenv()` works, using `os.environ.get()` with default values would provide more infor...
coderabbitai[bot] created a review comment on a pull request on Moohan/gartan_scraper_bot
_:hammer_and_wrench: Refactor suggestion_ **Enhance test assertions** The current assertion uses a generic key name and minimal validation. Consider adding more specific assertions to validate th...
coderabbitai[bot] created a review comment on a pull request on Moohan/gartan_scraper_bot
_:hammer_and_wrench: Refactor suggestion_ **Expand API endpoint test coverage** The current test only checks the happy path. Consider adding tests for error scenarios and different response types...
coderabbitai[bot] created a review comment on a pull request on Moohan/gartan_scraper_bot
_:warning: Potential issue_ **Security Risk: Avoid copying .env file into container image** Copying the .env file containing sensitive credentials into the container image poses a security risk, ...
coderabbitai[bot] created a review on a pull request on Moohan/gartan_scraper_bot
**Actionable comments posted: 8** <details> <summary>🧹 Outside diff range and nitpick comments (2)</summary><blockquote> <details> <summary>test_app.py (2)</summary><blockquote> Line range hint ...
coderabbitai[bot] created a review on a pull request on Moohan/gartan_scraper_bot
**Actionable comments posted: 8** <details> <summary>🧹 Outside diff range and nitpick comments (2)</summary><blockquote> <details> <summary>test_app.py (2)</summary><blockquote> Line range hint ...
coderabbitai[bot] created a comment on a pull request on Moohan/gartan_scraper_bot
<!-- This is an auto-generated comment: summarize by coderabbit.ai --> <!-- This is an auto-generated comment: skip review by coderabbit.ai --> > [!IMPORTANT] > ## Review skipped > > Draft detect...
Moohan pushed 3 commits to main Moohan/gartan_scraper_bot
Moohan closed a pull request on Moohan/gartan_scraper_bot
Create test_app.py
<!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Introduced unit tests for the `scrape_website` function and Flask API e...
coderabbitai[bot] created a review comment on a pull request on Moohan/gartan_scraper_bot
_:warning: Potential issue_ **Expand test coverage for scrape_website function** The current test implementation has several limitations: 1. The assertion uses a placeholder value 2. No error cas...
coderabbitai[bot] created a review comment on a pull request on Moohan/gartan_scraper_bot
_:hammer_and_wrench: Refactor suggestion_ **Enhance mock implementation for better test coverage** The current mock implementation could be more robust: 1. MockResponse should include standard re...
coderabbitai[bot] created a review comment on a pull request on Moohan/gartan_scraper_bot
_:warning: Potential issue_ **Enhance API endpoint test coverage** The current API test is basic and should include: 1. Error handling scenarios 2. Request header validation 3. Different response...
coderabbitai[bot] created a review comment on a pull request on Moohan/gartan_scraper_bot
_:hammer_and_wrench: Refactor suggestion_ **Consider enhancing the test client fixture** The current fixture is functional but could be improved by: 1. Adding test configuration 2. Setting up tes...
coderabbitai[bot] created a review on a pull request on Moohan/gartan_scraper_bot
**Actionable comments posted: 4** <details> <summary>📜 Review details</summary> **Configuration used: CodeRabbit UI** **Review profile: CHILL** <details> <summary>📥 Commits</summary> Files that...
coderabbitai[bot] created a review on a pull request on Moohan/gartan_scraper_bot
**Actionable comments posted: 4** <details> <summary>📜 Review details</summary> **Configuration used: CodeRabbit UI** **Review profile: CHILL** <details> <summary>📥 Commits</summary> Files that...