Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Removed no longer needed information about PHP 5.3#4854

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
weaverryan merged 1 commit intosymfony:2.3fromwouterj:backport_4753
Feb 24, 2015

Conversation

@wouterj
Copy link
Member

While merging,#4753, I thought we could remove these things completely from the docs, as PHP 5.3 and namespacing are quite common now.

QA
Doc fix?yes
New docs?no
Applies toall
Fixed tickets-

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

I think you can also remove the comma in front of the "and".

Copy link
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

I'm not entirely sure about this one. I think it's valid here to difference that this list is not "A, B or C", but instead "A, B or C, and D".

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Let's see what@weaverryan thinks.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

my "gut feeling" is that I agree with Wouter - it helps give the sentence some separation/pause. Is "gut feeling" good enough? ;)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

fair enough :)

@weaverryan
Copy link
Member

Yep, very very good idea. Not all things we say need to be there forever - it's cool how "new" things become commonplace. Thanks!

@weaverryanweaverryan merged commitb6636ae intosymfony:2.3Feb 24, 2015
weaverryan added a commit that referenced this pull requestFeb 24, 2015
This PR was merged into the 2.3 branch.Discussion----------Removed no longer needed information about PHP 5.3While merging,#4753, I thought we could remove these things completely from the docs, as PHP 5.3 and namespacing are quite common now.| Q   | A| --- | ---| Doc fix? | yes| New docs? | no| Applies to | all| Fixed tickets | -Commits-------b6636ae Removed no longer needed information about PHP 5.3
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@wouterj@weaverryan@xabbuh

[8]ページ先頭

©2009-2025 Movatter.jp