Skip to content

Conversation

NirantK
Copy link
Contributor

@NirantK NirantK commented Aug 19, 2025

Moved pyproject.toml to where it belongs. This led to some surprises, which are also handled in this PR.

  1. format docs and tests. They were excluded so far because of path issues as a silent error.
  2. Similar issues with pyright.

Important: I've excluded few things from pyright for now, fixed the rest.
We should remove exclusions. I'll leave that for @anistark

Fixes #2182

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Aug 19, 2025
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Aug 19, 2025
@NirantK NirantK requested review from anistark and jjmachan August 19, 2025 04:32
Copy link
Member

@jjmachan jjmachan left a comment

Choose a reason for hiding this comment

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

need some clarity on the specifics of the config

@NirantK NirantK marked this pull request as draft August 20, 2025 06:28
@NirantK NirantK closed this Aug 20, 2025
@NirantK NirantK reopened this Aug 20, 2025
@NirantK NirantK marked this pull request as ready for review August 20, 2025 14:07
Copy link

dosubot bot commented Aug 20, 2025

Related Documentation

No published documentation to review for changes on this repository.
Write your first living document

How did I do? Any feedback?  Join Discord

@jjmachan jjmachan merged commit 33ea219 into explodinggradients:main Aug 20, 2025
15 of 17 checks passed
ahgraber pushed a commit to ahgraber/ragas that referenced this pull request Aug 26, 2025
Moved pyproject.toml to where it belongs. This led to some surprises,
which are also handled in this PR.

1. format docs and tests. They were excluded so far because of path
issues as a silent error.
2. Similar issues with pyright.

Important: I've excluded few things from pyright for now, fixed the
rest.
We should remove exclusions. I'll leave that for @anistark

Fixes explodinggradients#2182
NirantK added a commit to NirantK/ragas-upgrades that referenced this pull request Sep 1, 2025
Moved pyproject.toml to where it belongs. This led to some surprises,
which are also handled in this PR.

1. format docs and tests. They were excluded so far because of path
issues as a silent error.
2. Similar issues with pyright. 

Important: I've excluded few things from pyright for now, fixed the
rest.
We should remove exclusions. I'll leave that for @anistark 

Fixes explodinggradients#2182
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate workspace.toml to pyproject.toml

2 participants