Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Are we there yet? I want to go PHP 5.4.... Are we there yet? #81

Closed
nickl- opened this issue Nov 26, 2012 · 9 comments
Closed

Are we there yet? I want to go PHP 5.4.... Are we there yet? #81

nickl- opened this issue Nov 26, 2012 · 9 comments
Milestone

Comments

@nickl-
Copy link
Member

nickl- commented Nov 26, 2012

I am sure we all know being irritating to our parents, repeatedly asking "are we there yet" when on long road trips, does this ring a bell? Some of us have already experienced this from the other end of the stick or will soon be. Myself and @alganet as far as I know, are not close to that so allow me to experience the child within as you will be hearing me say this frequently:

Are we there yet?

Since we've recently been prive to a release of 5.5.alpha1 is this a good enough excuse to bump the minimum requirement already? If you think it's too early still I have only one thing to say: with puppydog face...

But what about generators?

We can't exactly go skip a version PHP or can we? ...and I can't start irritating you with...

Are we there yet?

...for wanting to use yield and send and... anyway...

Are we there yet?

Would it be allright... is it acceptable... would anyone complain profusely or go tell the parents if I do? =)

Are we there yet?

I actually have a legitimate reason as argument, premise, good excuse to justify the move. One of the big gripes we had, amongst others, with implementing #28 was the fact that PHP had no support for status codes and while I have been thinking, even implemented some, alternative approaches to the array collection with code to reason mappings, as you may know PHP 5.4 came with support for:

http_response_code

Need I say more?I think the topic is well documented already. Now it's true we could polyfill backward compatibility but eeeuuww and what about array short syntax we can't exactly stub that compatible. The sooner we're done with 5.4 the sooner we can move to 5.5 \o/ so one more time:

Are we there yet, already?

@alganet
Copy link
Member

alganet commented Nov 27, 2012

I like the idea of moving to PHP 5.4 and 5.5. Many users still use 5.3 though, and it's still supported by Zend and umconfortably present in default setups like Ubuntu 12.04 LTS. I believe we should polish up the current releases (no new features, just making what's there more stable and documented) and release the first and last 5.3 stable release to honor our current 5.3 users!

Generators will make our pandas umprecedentaly awesome!

@alganet
Copy link
Member

alganet commented Dec 12, 2012

BTW I'm trying to migrate Rest to PSR-2 and complete legacy tests migration. Anyone willing to help this week?

@nickl-
Copy link
Member Author

nickl- commented Dec 12, 2012

I can help, only looking for work inbetween.

Do we really have to do full PSR-2? =/

I started writing the code conventions if I recall where did I go with that?...

@alganet
Copy link
Member

alganet commented Dec 12, 2012

I personally don't like PSR-2, and even though adopting it isn't a sign of quality, users tend to interpret adoption of PSRs as such.

During the PSR-2fication I've found that most of our code (except for the 80 column limit and brances on control structures) is already OK.

I've also added docblocks to the code I've fixed, which is another popular "sign of quality" (even Ohloh considers that when crawling projects).

We still need our own conventions for naming, coupling, dependency injection and so on =)

@nickl-
Copy link
Member Author

nickl- commented Dec 12, 2012

Afaik the spec allows for 120 chars soft limit
There are several other items that are optional so you can still claim PSR-2 with exclusions

I really really really don't want begin/end for one line statements it's going against PHP design and what we stand for arguably. It wastes so much useless space =(

Wheres that post...

@nickl-
Copy link
Member Author

nickl- commented Dec 12, 2012

Found it https://github.com/Respect/project-info/wiki/Coding-Standards

It still needs a little work it seems.

@alganet
Copy link
Member

alganet commented Dec 12, 2012

I agree on letting the optional PSR-2 items out. I'm not sure about the 80/120 column limit, we need to see how the tools for checking that behave. In Zend, they had a hard and soft limit but the PHP CodeSniffer printed warnings when you break the hard limit. I should have tested that for PSR-2 =(

@nickl-
Copy link
Member Author

nickl- commented Dec 12, 2012

Sorry man that wiki page I found was your coding standard write-up

This is what I was doing.
Respect Syntax Convention (RSC)

from PSR-2
There MUST NOT be a hard limit on line length; the soft limit MUST be 120 characters; lines SHOULD be 80 characters or less.

The tools, both of them, were still very dodgy when I last looked at them and we will likely need to work on them to get it fixed. We might as well customise it to our RSC then where needed, what do you think?

@augustohp
Copy link
Member

I am closing this, although I am always eager to move forward into version requirements I would not likely do so just for the sake of it.

If we ever need to update the version, we should do so in that occasion.

@augustohp augustohp added this to the 1.0 milestone May 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants