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.

Sign up for GitHub

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

Jump to bottom

refactor: Remove Document.__str__() #6155

Closed
wants to merge 1 commit into from
Closed

refactor: Remove Document.__str__() #6155

wants to merge 1 commit into from

Conversation

silvanocerza
Copy link
Contributor

Proposed Changes:

Remove Document.__str__() as it's not necessary.

How did you test it?

Ran tests locally.

Notes for the reviewer

N/A

Checklist

@silvanocerza silvanocerza added the 2.x Related to Haystack v2.0 label Oct 23, 2023
@silvanocerza silvanocerza self-assigned this Oct 23, 2023
@silvanocerza silvanocerza requested review from a team as code owners October 23, 2023 14:40
@silvanocerza silvanocerza requested review from dfokina and ZanSara and removed request for a team October 23, 2023 14:40
@silvanocerza silvanocerza deleted the document-str-remove branch October 23, 2023 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dfokina dfokina Awaiting requested review from dfokina dfokina is a code owner automatically assigned from deepset-ai/documentation

@ZanSara ZanSara Awaiting requested review from ZanSara ZanSara is a code owner automatically assigned from deepset-ai/open-source-engineering

Assignees

@silvanocerza silvanocerza

Labels
2.x Related to Haystack v2.0 topic:tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@silvanocerza