Skip to content
This repository has been archived by the owner on Jun 1, 2024. It is now read-only.

Parallization (DontAwaitInALoopLinter) #6

Open
fredemmott opened this issue Jun 21, 2018 · 3 comments
Open

Parallization (DontAwaitInALoopLinter) #6

fredemmott opened this issue Jun 21, 2018 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@fredemmott
Copy link
Contributor

We probably want to run tests within a file sequentially (cc @BobertForever ?) - but we can probably run multiple files simultaneously.

These are picked up by the DontAwaitInALoop linter - adding HHAST_IGNORE_ERROR[DontAwaitInALoop] in the places this matters

@fredemmott fredemmott added the enhancement New feature or request label Jun 29, 2018
@fredemmott
Copy link
Contributor Author

Major options:

  • do everything sequentially
  • use async
  • spin up HHVM in CLI server mode (or HTTP on a unix socket), and run separate classes in separate requests contexts
  • if xbox is usable in CLI mode, this might be a simpler alternative

This is an extension beyond the basic requirements.

@fredemmott
Copy link
Contributor Author

XBox is usable in CLI mode

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants
@fredemmott @rob0rt @wlin53 and others