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

parallelize WPT tests run, add bandit linter, and misc changes #828

Merged
merged 8 commits into from
Jun 16, 2023

Conversation

thiagowfx
Copy link
Contributor

@thiagowfx thiagowfx commented Jun 6, 2023

With the --processes this change:

  • mapper-headless: 13min
  • chromedriver-headless: 15min

Without this change:

  • mapper-headless: 16min
  • chromedriver-headless: 15min

The impact isn't that great, but it doesn't make things worse either.

@thiagowfx thiagowfx force-pushed the thiagowfx/next branch 5 times, most recently from ae013dc to 4731b0a Compare June 12, 2023 09:22
@thiagowfx thiagowfx changed the title (evergreen) misc changes parallelize WPT tests run, add bandit linter, and misc changes Jun 13, 2023
@thiagowfx thiagowfx requested a review from sadym-chromium June 13, 2023 13:04
@thiagowfx thiagowfx marked this pull request as ready for review June 13, 2023 13:04
@thiagowfx thiagowfx requested a review from sadym-chromium June 16, 2023 12:36
@thiagowfx thiagowfx enabled auto-merge (squash) June 16, 2023 12:36
Copy link
Collaborator

@sadym-chromium sadym-chromium left a comment

Choose a reason for hiding this comment

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

Ooops. I thought I pressed lgtm.

@thiagowfx thiagowfx merged commit 8569d54 into main Jun 16, 2023
@thiagowfx thiagowfx deleted the thiagowfx/next branch June 16, 2023 20:18
@OrKoN
Copy link
Collaborator

OrKoN commented Jun 21, 2023

nproc is not available on Mac. Let's not write new shell scripts from now on in this repo, WDYT?

@thiagowfx
Copy link
Contributor Author

x-ref #876

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.

3 participants