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

Allow npm and yarn support for testing. #142

Open
infinity7998 opened this issue Jan 22, 2021 · 0 comments
Open

Allow npm and yarn support for testing. #142

infinity7998 opened this issue Jan 22, 2021 · 0 comments

Comments

@infinity7998
Copy link

Consider the scripts in the root package.json:

{
"scripts": {
    "test:auth": "cd modules/auth && yarn test",
    "test:database": "cd modules/database && yarn test",
    "test:firestore": "cd modules/firestore && yarn test",
    "test": "yarn test:auth && yarn test:database && yarn test:firestore"
  },
}

Problem

If someone does not have yarn installed and works exclusively with npm then all the scripts FAIL.

Suggested fix

Probably a better idea to either default to npm or else have the keys say : test:auth:npm and test:auth:yarn, and handle the scripts accordingly.

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

No branches or pull requests

1 participant