[Phpmyadmin-devel] Coding style for error reporting server

[Michal Čihař]

Hi

Dne Tue, 30 Jul 2013 11:47:57 +0200
Mohamed Ashraf <mohamed.ashraf.213 at gmail.com> napsal(a):

> On Tue, Jul 30, 2013 at 11:39 AM, Michal Čihař <michal at cihar.com> wrote:
> > Dne Tue, 30 Jul 2013 11:37:51 +0200
> > Michal Čihař <michal at cihar.com> napsal(a):
> >
> >> Hi
> >>
> >> Dne Tue, 30 Jul 2013 11:31:15 +0200
> >> Mohamed Ashraf <mohamed.ashraf.213 at gmail.com> napsal(a):
> >>
> >> > I was not exactly following any coding guidelines other than my own.
> >> > However most of the warnings are because I use spaces and the
> >> > convention is to use tabs. I can write a vim regex to change the
> >> > files.
> >> >
> >> > I believe that the CakePHP checkstyle should be used since while this
> >> > is part of the phpmyadmin family of webapps we should respect the
> >> > guidelines of the used framework so that we are not fighting an uphill
> >> > battle with our own framework over code guidelines.
> >> >
> >> > Also what checkstyle guidelines are you using currently phpmyadmin or cakephp.
> >>
> >> Currently it's using CakePHP style.
> >
> > ... you can actually see or change this in build.xml
> 
> who has commit access to the repo other than you? I was wondering if
> you could add one or both my mentors on this list so that they can
> accept my pull requests.

Any phpMyAdmin developer should have access, so they should be able to
do that (in case it does not work, I've done something wrong and please
let me know).

-- 
	Michal Čihař | http://cihar.com | http://blog.cihar.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.phpmyadmin.net/pipermail/developers/attachments/20130730/b29cbf49/attachment.sig>