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

Consolidate test sample documents into one subdir #356

Merged
merged 1 commit into from
Nov 11, 2024

Conversation

bbrowning
Copy link
Contributor

This gets rid of the extra testdata directory for functional tests and collapses those into a single one, using the existing sample_documents subdir of the unit tests.

This gets rid of the extra testdata directory for functional tests
and collapses those into a single one, using the existing
sample_documents subdir of the unit tests.

Signed-off-by: Ben Browning <[email protected]>
@mergify mergify bot added documentation Improvements or additions to documentation testing Relates to testing labels Nov 8, 2024
@bbrowning
Copy link
Contributor Author

This addresses some review comments from @khaledsulayman on #349 that I wasn't able to get in before that one auto-merged. Not urgent, and just some cleanup of folder structures for the new functional test sample docs.

Copy link
Member

@khaledsulayman khaledsulayman left a comment

Choose a reason for hiding this comment

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

Thanks for this change, Ben! LGTM!

@mergify mergify bot added the one-approval label Nov 11, 2024
@mergify mergify bot merged commit 9173237 into instructlab:main Nov 11, 2024
24 checks passed
@mergify mergify bot removed the one-approval label Nov 11, 2024
@bbrowning bbrowning deleted the chunking_testing branch November 11, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation testing Relates to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants