Hi Torsten,

I like all this cleanups and recategorizations.

But I have two requests, tell me what you think:

 - first and more important. I'd like that we define a convention document
in a wiki page or pillar document for protocols. What are the correct
protocols to use and why? This is important for newcomers and for making
tools. I see that you're doing a lot of recategorizations but I'd like if
we set them up and discuss them in the mailing list.

 - second and more technological. Why are all categorizations splitted in
mini pull requests, scoped to one class? Can we group them at least by
package? otherwise this is killing our integration infrastructure (which is
modest but for free :)). Also separating all this changes means that we
cannot easily find them in the history later on, I'd prefer that we group
related changes in related commits if possible.  Marcus was telling me this
may be because people have trouble reviewing them if they are too big? I
can review if you put me ask me as reviewer.

Tx,
Guilel

On Sun, Mar 25, 2018 at 8:54 PM, Astares <notificati...@github.com> wrote:

> https://pharo.fogbugz.com/f/cases/21646/AbstractFileReference-should-
> use-utilities-protocol-name
> ------------------------------
> You can view, comment on, or merge this pull request online at:
>
>   https://github.com/pharo-project/pharo/pull/1149
> Commit Summary
>
>    - 21646 AbstractFileReference should use "utilities" protocol name
>
> File Changes
>
>    - *M* src/FileSystem-Core/AbstractFileReference.class.st
>    <https://github.com/pharo-project/pharo/pull/1149/files#diff-0> (2)
>
> Patch Links:
>
>    - https://github.com/pharo-project/pharo/pull/1149.patch
>    - https://github.com/pharo-project/pharo/pull/1149.diff
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/pharo-project/pharo/pull/1149>, or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AArO4g3P9uj62nQGbRG-T842HCGZZHt7ks5th-f1gaJpZM4S6QZ_>
> .
>



-- 



Guille Polito

Research Engineer

Centre de Recherche en Informatique, Signal et Automatique de Lille

CRIStAL - UMR 9189

French National Center for Scientific Research - *http://www.cnrs.fr
<http://www.cnrs.fr>*


*Web:* *http://guillep.github.io* <http://guillep.github.io>

*Phone: *+33 06 52 70 66 13

Reply via email to