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

fix(llama-index): Follow naming convention of private attributes #971

Merged
merged 7 commits into from
Aug 27, 2024

Conversation

joelbarmettlerUZH
Copy link
Contributor

@joelbarmettlerUZH joelbarmettlerUZH commented Aug 27, 2024

Fixes #970
Fixes #955

@joelbarmettlerUZH joelbarmettlerUZH requested a review from a team as a code owner August 27, 2024 06:26
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Aug 27, 2024
Copy link
Contributor

github-actions bot commented Aug 27, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@joelbarmettlerUZH
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Aug 27, 2024
@axiomofjoy
Copy link
Contributor

Thank you @joelbarmettlerUZH. I'll take a look tomorrow.

@axiomofjoy axiomofjoy self-requested a review August 27, 2024 08:11
@RogerHYang
Copy link
Contributor

RogerHYang commented Aug 27, 2024

@joelbarmettlerUZH you can fix the formatting by running tox in the python folder as follows.

tox run -e ruff-mypy-test-llama_index-latest

This requires installing tox-uv as follows.

pip install tox-uv

See here if you need to install uv.

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Aug 27, 2024
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Aug 27, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 27, 2024
@RogerHYang RogerHYang merged commit ba203b8 into Arize-ai:main Aug 27, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug] llama-index canary failing [bug] Support for LlamaIndex with Pydantic2
4 participants