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 question utils for enhanced performance and readability #10551

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

djeada
Copy link
Contributor

@djeada djeada commented Jan 9, 2025

  • Renamed p to README_PATH for clarity.
  • Introduced EXERCISES_PATH for the exercises directory.
  • Precompiled regex patterns for efficiency.
  • Changed get_file_list to get_file_content and made it return a string directly.
  • Used UTF-8 encoding when opening files.
  • Simplified get_question_list and get_answered_questions by using list comprehensions and conditional checks.
  • Updated type hints for better clarity.
  • Removed unused variables and optimized function logic.
  • Ensured all functions read the file content once when needed.
  • Enhanced the docstring example for accuracy.
  • Maintained all original functionalities and interfaces for backward compatibility.

…tended functionality while preserving backward compatibility
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