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

Run tests in parallel #60

Merged
merged 2 commits into from
Jul 11, 2024
Merged

Run tests in parallel #60

merged 2 commits into from
Jul 11, 2024

Conversation

matheper
Copy link
Contributor

@matheper matheper commented Jul 10, 2024

  • Fix a small bug when calling the library Engine directly in tests (it was creating a path that includes the prefix local:)
  • Add docstring to BackendEngine warning to not use the class directly and that repo_id should not include prefix (it works correctly when called form LibraryExpert classes)
  • Running pytest using 4 processes (max n. processes for public repos). Doesn't seem to help much but doesn't make it worse either. I will keep the 4 proccesses config
  • black / isort changes (I am creating a second PR setting up an action to run them automatically)

@sordonia sordonia merged commit 93d86da into main Jul 11, 2024
4 checks passed
@sordonia sordonia deleted the test-tests-parallel branch July 11, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants