PR without Tests

Hello,
   Pharo is not Smalltalk / Smalltalk is not Pharo / Pharo is Smalltalk.
Now that I have your attention because it seems the only read mail has
to have this.

We have to continue improving the quality of Pharo. One of the biggest
assets that we have in Pharo are the tons of tests. So, we need to
reject any PR that does not have tests or it is not already tested.

To ease this quality measure, please add some comments on the PR about
how your new code or modified code is tested or benchmarked.

Cheers.

--
Pablo Tesone.
[hidden email]

Of course, the already open PR are there, and it is ok, we can handle them.

On Thu, Feb 6, 2020 at 10:29 AM [hidden email] <[hidden email]> wrote:


--
Pablo Tesone.
[hidden email]

+1000

Great news!

I was wondering if they were also something planned to get some coverage information report to help to add more tests on parts of the system that are not tested?

Cheers,
Vincent


-----Original Message-----
From: Pharo-dev On Behalf Of [hidden email]
Sent: Thursday, 6 February 2020 10:30
To: Pharo-dev <[hidden email]>; [hidden email] list <[hidden email]>
Subject: [Pharo-dev] PR without Tests

Hello,
   Pharo is not Smalltalk / Smalltalk is not Pharo / Pharo is Smalltalk.
Now that I have your attention because it seems the only read mail has to have this.

We have to continue improving the quality of Pharo. One of the biggest assets that we have in Pharo are the tons of tests. So, we need to reject any PR that does not have tests or it is not already tested.

To ease this quality measure, please add some comments on the PR about how your new code or modified code is tested or benchmarked.

Cheers.

--
Pablo Tesone.
[hidden email]