Skip to content

Conversation

@rvhonorato
Copy link
Member

@rvhonorato rvhonorato commented Apr 15, 2025

===autogenerated description===

This pull request includes multiple changes to improve the testing framework, update the documentation, and refactor the codebase for better performance and readability. The most important changes are summarized below:

Testing Framework Improvements:

  • .github/workflows/ci.yml: Separated unit tests and integration tests, added detailed pytest options, and included hypothesis statistics.
  • pyproject.toml: Added a marker for integration tests to the pytest configuration.

Documentation Updates:

  • README.md: Updated usage instructions, added examples for running prodigy on single and multiple structures, and improved the help message format. [1] [2] [3] [4] [5]

Codebase Refactoring:

These changes collectively enhance the project's maintainability, usability, and testing capabilities.

@rvhonorato rvhonorato requested a review from Copilot April 15, 2025 13:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

tests/test_prodigy.py:226

  • The variable 'diffs' is referenced without being initialized, which will result in a runtime error during test execution. Consider initializing 'diffs' (e.g., as an empty dictionary with lists for each key) before iterating over it.
for k in diffs.keys():

@rvhonorato rvhonorato self-assigned this Apr 15, 2025
@rvhonorato rvhonorato marked this pull request as ready for review April 15, 2025 13:26
@rvhonorato rvhonorato merged commit f7db1ce into main Apr 15, 2025
7 checks passed
@rvhonorato rvhonorato deleted the batch branch April 15, 2025 13:34
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.

2 participants