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

refactor/debug #377

Merged
merged 17 commits into from
Apr 26, 2024
Merged

refactor/debug #377

merged 17 commits into from
Apr 26, 2024

Conversation

gnikit
Copy link
Member

@gnikit gnikit commented Apr 26, 2024

  • refactor: debug CLI to smaller functinos
  • refactor: convert format to f-strings
  • refactor: made functions to ensure debug_filepath exists and is accessible
  • refactor: renamed variables
  • refactor: chang
  • refactor: minor conditional in workspace symbols
  • chore: reformated debug_diagnostics message
  • feat: add debug exceptions
  • refactor: removed unnecessary levels of intendation
  • refactor: add utf-8 encoding to file open in debug_server_parser
  • refactor: remove unnecessary code in debug_server_parser
  • refactor: improve debug_server_parser code readability and consistency
  • refactor: move config file read to inner method for debug_server_parser
  • refactor: moved debug CLI code to a dedicated module
  • refactor: swap json with json5 package for debugging
  • refactor: improve code reusability in debug module
  • refactor: rename debug_server_parser to debug_parser

Copy link

codecov bot commented Apr 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.66%. Comparing base (522307d) to head (13abbce).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #377   +/-   ##
=======================================
  Coverage   87.66%   87.66%           
=======================================
  Files          35       35           
  Lines        4793     4793           
=======================================
  Hits         4202     4202           
  Misses        591      591           

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

@gnikit gnikit merged commit de67ae5 into master Apr 26, 2024
36 checks passed
@gnikit gnikit deleted the refactor/debug branch April 26, 2024 00:46
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.

1 participant