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

Add --test-script flag to run custom tests before publish #557

Merged
merged 33 commits into from Aug 9, 2020
Merged

Add --test-script flag to run custom tests before publish #557

merged 33 commits into from Aug 9, 2020

Conversation

dopecodez
Copy link
Collaborator

@dopecodez dopecodez commented Jul 17, 2020

Fixes #543 . Updated readme as per the discussion on this issue.

@dopecodez dopecodez changed the title Implent --test-script flag to run custom tests before publish Implement --test-script flag to run custom tests before publish Jul 17, 2020
@sindresorhus sindresorhus changed the title Implement --test-script flag to run custom tests before publish Implement --test-script flag to run custom tests before publish Jul 17, 2020
@sindresorhus sindresorhus changed the title Implement --test-script flag to run custom tests before publish Add --test-script flag to run custom tests before publish Jul 17, 2020
source/cli.js Outdated Show resolved Hide resolved
readme.md Outdated Show resolved Hide resolved
readme.md Outdated Show resolved Hide resolved
source/index.js Outdated Show resolved Hide resolved
source/cli.js Outdated Show resolved Hide resolved
readme.md Outdated Show resolved Hide resolved
@dopecodez
Copy link
Collaborator Author

Updated Custom tests to User Defined Tests wherever applicable, and other PR comments. Could you take a look @sindresorhus ?

readme.md Outdated Show resolved Hide resolved
readme.md Outdated Show resolved Hide resolved
readme.md Outdated Show resolved Hide resolved
readme.md Outdated Show resolved Hide resolved
source/cli.js Outdated Show resolved Hide resolved
dopecodez and others added 4 commits July 26, 2020 17:06
Co-authored-by: Sindre Sorhus <sindresorhus@gmail.com>
Co-authored-by: Sindre Sorhus <sindresorhus@gmail.com>
readme.md Outdated Show resolved Hide resolved
Co-authored-by: Sindre Sorhus <sindresorhus@gmail.com>
@sindresorhus
Copy link
Owner

Can you fix the conflict?

@dopecodez
Copy link
Collaborator Author

Fixed the conflict @sindresorhus

readme.md Outdated Show resolved Hide resolved
source/cli.js Outdated Show resolved Hide resolved
@sindresorhus sindresorhus merged commit 0c46a2f into sindresorhus:master Aug 9, 2020
@sindresorhus
Copy link
Owner

Thanks for the PR. 👍🏻 It would be great if next time, you could look over the diff a couple of times after doing changes. I feel like I commented on a lot of obvious stuff that could have been caught by you.

@dopecodez
Copy link
Collaborator Author

I agree @sindresorhus , just got bogged down with work. Will up my PR game for sure.

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.

Specify test command to run
2 participants