Skip to content

Conversation

LVivona
Copy link

@LVivona LVivona commented Jul 22, 2025

This change separates the custom exception definition from the config_parser module. It improves code organization and aligns with single-responsibility principles by moving exception-related logic to its own exception.py file, making both modules easier to maintain and extend.


PR Type

[x] Refactor
[ ] Feature
[ ] Fix
[ ] Documentation
[ ] Other


Short Description

Refactored code by relocating exception handling from config_parser.py to a new module exception.py.


ClickUp Ticket(s)

N/A


Tests Added

  • No new tests added; existing functionality preserved.
  • Exception import paths updated where necessary.

LVivona and others added 4 commits July 22, 2025 01:05
This change separates the custom exception definition from the config_parser module. It improves code organization and aligns with single-responsibility principles.
@codecov-commenter
Copy link

codecov-commenter commented Jul 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.13%. Comparing base (484049c) to head (fe42fa4).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #210   +/-   ##
=======================================
  Coverage   93.13%   93.13%           
=======================================
  Files          32       33    +1     
  Lines        3596     3597    +1     
  Branches      184      263   +79     
=======================================
+ Hits         3349     3350    +1     
  Misses        247      247           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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