Skip to content
This repository was archived by the owner on Sep 18, 2025. It is now read-only.

Move simuleval/tests to top level#60

Open
mandyschen wants to merge 2 commits intomainfrom
tests_to_top_level
Open

Move simuleval/tests to top level#60
mandyschen wants to merge 2 commits intomainfrom
tests_to_top_level

Conversation

@mandyschen
Copy link
Contributor

No description provided.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 1, 2023
@mandyschen mandyschen marked this pull request as draft August 1, 2023 18:44
@mandyschen mandyschen marked this pull request as ready for review August 1, 2023 18:49
Copy link
Contributor

@annasun28 annasun28 left a comment

Choose a reason for hiding this comment

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

Thanks Mandy!

@yairz-q
Copy link

yairz-q commented Nov 19, 2024

can you please consider merging this and also moving pytest out from the requires?
https://stackoverflow.com/questions/28509965/setuptools-development-requirements

you can put it in the extra_require

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants