Skip to content
This repository has been archived by the owner on Dec 7, 2022. It is now read-only.

Add basic unit tests policy #3411

Merged
merged 1 commit into from Apr 6, 2018

Conversation

bmbouter
Copy link
Member

@bmbouter bmbouter commented Apr 5, 2018

This policy is a starting point based feedback via pulp-dev. The
language adopted was shared via pulp-dev for comment.

Copy link
Contributor

@asmacdo asmacdo left a comment

Choose a reason for hiding this comment

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

+1

Unit Tests
==========

All new code is highly encouraged to have basic unit tests that demonstrate its functionality. A
Copy link
Contributor

Choose a reason for hiding this comment

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

s/All new/New/

@bmbouter
Copy link
Member Author

bmbouter commented Apr 5, 2018

Thanks @asmacdo, I fixed it

Copy link
Contributor

@asmacdo asmacdo left a comment

Choose a reason for hiding this comment

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

I approve, but I hope others will review also since there were a lot of opinions.

@daviddavis
Copy link
Contributor

daviddavis commented Apr 6, 2018

Btw, it looks like you have a commit you need to squash.

This policy is a starting point based feedback via pulp-dev. The
language adopted was shared via pulp-dev for comment.
@bmbouter bmbouter force-pushed the add-unit-test-policy-to-docs branch from 02f1f4d to 8cc7752 Compare April 6, 2018 13:56
@bmbouter
Copy link
Member Author

bmbouter commented Apr 6, 2018

Why yes I did. Thank you.

@mibanescu
Copy link
Member

Looks good to me.

@bmbouter bmbouter merged commit ff093d4 into pulp:3.0-dev Apr 6, 2018
@bmbouter bmbouter deleted the add-unit-test-policy-to-docs branch April 6, 2018 18:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
5 participants