[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