← Back to team overview

elementary-dev-community team mailing list archive

Re: How to review and merge branches

 

How do you figure? The go language community uses one and they rave about
it. We use them at work (c++) as well and its uses an obnoxious style, but
it's still more readable than a dozen different conventions.
On Mar 31, 2013 5:39 AM, "Sergey "Shnatsel" Davidoff" <
sergey@xxxxxxxxxxxxxxxx> wrote:

> I'm afraid automatic "prettifiers" are a terrible idea because blindly
> restyling the code usually makes it lose any remains of readability it used
> to have. In other words, automatically restyled code is even less readable
> than code with a foreign coding style.
>
>
> 2013/3/31 David Gomes <david@xxxxxxxxxxxxxxxx>
>
>> I wrote this in order to check for code style errors, but it's not
>> perfect it's just a help-tool:
>>
>> https://github.com/elementary/vala-analyzer
>>
>> We have 'considered' using a prettifier too, but I just use Emacs to fix
>> some stuff on my code - a prettifier script would be too much work and I
>> don't know of any libraries that would help me with the task.
>>
>>
>> On Sun, Mar 31, 2013 at 3:34 AM, Craig <weberc2@xxxxxxxxx> wrote:
>>
>>> Good work David. Have you (elementary) considered using a prettifier to
>>> standardize a code style upon pushing to your trunk?
>>>  On Mar 28, 2013 7:17 PM, "Cody Garver" <cody@xxxxxxxxxxxxxxxx> wrote:
>>>
>>>> Cool, it's pretty thorough.
>>>>
>>>>
>>>> On Wed, Mar 27, 2013 at 7:58 AM, David Gomes <david@xxxxxxxxxxxxxxxx>wrote:
>>>>
>>>>> http://dl.dropbox.com/u/19899464/reviewstutorial.html
>>>>>
>>>>> Hello guys,
>>>>>
>>>>> From time to time somebody still has doubts on how to use Launchpad
>>>>> and Bazaar to review and merge branches to trunk so I wrote a tutorial.
>>>>> Note though that it may need expansion.
>>>>>
>>>>> Many times, even experienced developers who have been in the Apps Team
>>>>> for a long time make mistakes so even if you already know how to do it,
>>>>> reading the tutorial won't hurt.
>>>>>
>>>>> I also recommend that all developers that in the future are to join
>>>>> the Apps Team read this several times because even though we can always
>>>>> revert messed-up commits, it's better to do it right at the first time.
>>>>>
>>>>> Best regards,
>>>>> David "Munchor" Gomes
>>>>>
>>>>> --
>>>>> Mailing list: https://launchpad.net/~elementary-dev-community
>>>>> Post to     : elementary-dev-community@xxxxxxxxxxxxxxxxxxx
>>>>> Unsubscribe : https://launchpad.net/~elementary-dev-community
>>>>> More help   : https://help.launchpad.net/ListHelp
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> Cody Garver
>>>>
>>>> --
>>>> Mailing list: https://launchpad.net/~elementary-dev-community
>>>> Post to     : elementary-dev-community@xxxxxxxxxxxxxxxxxxx
>>>> Unsubscribe : https://launchpad.net/~elementary-dev-community
>>>> More help   : https://help.launchpad.net/ListHelp
>>>>
>>>>
>>
>> --
>> Mailing list: https://launchpad.net/~elementary-dev-community
>> Post to     : elementary-dev-community@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~elementary-dev-community
>> More help   : https://help.launchpad.net/ListHelp
>>
>>
>
>
> --
> Sergey "Shnatsel" Davidoff
> OS architect @ elementary
>

Follow ups

References