PR without Tests

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

PR without Tests

tesonep@gmail.com
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]

Reply | Threaded
Open this post in threaded view
|

Re: PR without Tests

tesonep@gmail.com
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:

>
> 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]



--
Pablo Tesone.
[hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: [rmod] PR without Tests

EstebanLM
In reply to this post by tesonep@gmail.com
+1000

On 6 Feb 2020, at 10:29, [hidden email] wrote:


-- 

Reply | Threaded
Open this post in threaded view
|

Re: PR without Tests

Pharo Smalltalk Developers mailing list
In reply to this post by tesonep@gmail.com
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]