[Sugar-devel] [sugarlabs/sugar] Failure to merge PR #327

Tony Anderson tony_anderson at usa.net
Wed Apr 26 03:40:47 EDT 2017


The purpose of the feature has been described over and over again. It is 
not about files with blank titles. Utkarsh chose to alert the user who 
clicked on ok with a blank title. Perfectly reasonable programming decision.

The technical issues you raised were fixed by Utkarsh quickly and 
apparently to your satisfaction. Your refusal to merge the PR is now 
based sole on your personal prejudices. There is a setting so that the 
feature by default is not enabled, so it is really difficult to 
understand your concerns.

You do not agree with it. Walter said go ahead. Sebastian says go ahead. 
Utkarsh says go ahead. I say go ahead.

Tony






On 04/26/2017 02:58 PM, James Cameron wrote:
>
> Well, we're hearing a lot from you, but I'm hoping other people will 
> say what they think. I'll hold off on #327 
> <https://github.com/sugarlabs/sugar/pull/327> some more now that I can 
> see there is not general agreement on the purpose of the feature; you 
> saying it is to avoid blank or empty titles, and me saying it is to 
> force reflection and choice of a title.
>
>> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub 
> <https://github.com/sugarlabs/sugar/pull/752#issuecomment-297258026>, 
> or mute the thread 
> <https://github.com/notifications/unsubscribe-auth/AAULkg9ZAaqchH9FlmczI423FmZZUu3Dks5rzusagaJpZM4NIV2L>.
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20170426/ac513a0c/attachment.html>


More information about the Sugar-devel mailing list