pull requests: variable names
Nick Coghlan
ncoghlan at gmail.com
Sat Feb 14 23:50:53 EST 2015
On 15 February 2015 at 06:36, Thomas De Schampheleire
<patrickdepinguin at gmail.com> wrote:
> Hi,
>
> In the pull request code (both controllers and templates) I find the
> variable names non-intuitive and it is therefore harder to make
> changes.
>
> With this mail I would like to get an agreement on the terminology we
> can change to.
>
> - org vs other --> source/src and dest
> - c.cs_repo: what does 'cs' stand for here? This could become src_repo
> - c.a_repo: what does 'a' stand for here?? This could become dest_repo.
>
> Other variable names with the same prefixes 'cs'/'a'/'org'/'other'
> would be changed too, of course.
+1 from me for the general idea of having PR code focus on "src ->
dest", without making assumptions about the "src" being outside the
organisation and the "dest" being inside.
Regards,
Nick.
--
Nick Coghlan | ncoghlan at gmail.com | Brisbane, Australia
More information about the kallithea-general
mailing list