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

Release v2.3.42 #18916

Merged
merged 3 commits into from May 30, 2016

Conversation

Projects
None yet
8 participants
@fabpot
Member

fabpot commented May 30, 2016

Changes since last release: v2.3.41...7143b29

Changelog

@fabpot fabpot merged commit 0baa58d into symfony:2.3 May 30, 2016

0 of 3 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
fabbot.io Doing some proofreading and checking your coding style.
Details

@fabpot fabpot deleted the fabpot:release-2.3.42 branch May 30, 2016

@aivus

This comment has been minimized.

Show comment
Hide comment
@aivus

aivus May 31, 2016

Contributor

It seems #18908 reverted but exists in your list.

Contributor

aivus commented May 31, 2016

It seems #18908 reverted but exists in your list.

@sstok

This comment has been minimized.

Show comment
Hide comment
@sstok

sstok May 31, 2016

Contributor

@aivus It's not a problem, I checked and there are no security problems introduced with the change.

Contributor

sstok commented May 31, 2016

@aivus It's not a problem, I checked and there are no security problems introduced with the change.

@aivus

This comment has been minimized.

Show comment
Hide comment
@aivus

aivus May 31, 2016

Contributor

@sstok yeah, but this fix doesn't included to release.

Contributor

aivus commented May 31, 2016

@sstok yeah, but this fix doesn't included to release.

@nicolas-grekas

This comment has been minimized.

Show comment
Hide comment
@nicolas-grekas

nicolas-grekas May 31, 2016

Member

changelog updated in PR description

Member

nicolas-grekas commented May 31, 2016

changelog updated in PR description

@sstok

This comment has been minimized.

Show comment
Hide comment
@sstok

sstok May 31, 2016

Contributor

Oh OK 😄

Contributor

sstok commented May 31, 2016

Oh OK 😄

@chalasr

This comment has been minimized.

Show comment
Hide comment
@chalasr

chalasr Jun 9, 2016

Member
- - Robin Chalas (chalas_r)
+ - Robin Chalas (chalasr)
Member

chalasr commented on CONTRIBUTORS.md in b4a9d25 Jun 9, 2016

- - Robin Chalas (chalas_r)
+ - Robin Chalas (chalasr)

This comment has been minimized.

Show comment
Hide comment
@javiereguiluz

javiereguiluz Jun 9, 2016

Member

This is generated automatically. Did you change your username recently?

Member

javiereguiluz replied Jun 9, 2016

This is generated automatically. Did you change your username recently?

This comment has been minimized.

Show comment
Hide comment
@chalasr

chalasr Jun 9, 2016

Member

Nope, but this is the one of my sensiolabs connect account. Maybe it's the origin of the typo ? (BTW I would like to change my username on connect but it seems impossible)

Member

chalasr replied Jun 9, 2016

Nope, but this is the one of my sensiolabs connect account. Maybe it's the origin of the typo ? (BTW I would like to change my username on connect but it seems impossible)

This comment has been minimized.

Show comment
Hide comment
@wouterj

wouterj Jun 9, 2016

Member

The username shown here is your connect account username and not the github user account. It might have to be explicitely described at the top of the file, as it has confused some others as well.

Member

wouterj replied Jun 9, 2016

The username shown here is your connect account username and not the github user account. It might have to be explicitely described at the top of the file, as it has confused some others as well.

This comment has been minimized.

Show comment
Hide comment
@chalasr

chalasr Jun 9, 2016

Member

Thank you @wouterj

Member

chalasr replied Jun 9, 2016

Thank you @wouterj

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment