Skip to content

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
@dopecodez dopecodez requested a review from sindresorhus July 18, 2020 06:47
@dopecodez dopecodez requested a review from sindresorhus July 23, 2020 15:32
@dopecodez
Copy link
Collaborator Author

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

@dopecodez dopecodez requested a review from sindresorhus July 25, 2020 11:37
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>
@dopecodez dopecodez requested a review from sindresorhus July 26, 2020 13:11
Co-authored-by: Sindre Sorhus <sindresorhus@gmail.com>
@dopecodez dopecodez requested a review from sindresorhus July 27, 2020 06:17
@sindresorhus
Copy link
Owner

Can you fix the conflict?

@dopecodez
Copy link
Collaborator Author

Fixed the conflict @sindresorhus

@dopecodez dopecodez requested a review from sindresorhus August 6, 2020 10:49
@dopecodez dopecodez requested a review from sindresorhus August 9, 2020 13:45
@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