NotificationsYou must be signed in to change notification settings
Fork9.7k
Star30.8k

Added missing headers in fixture files#21171

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

Jump to bottom

Merged

nicolas-grekas merged 1 commit intosymfony:3.1fromlinaori:fix/fixture-header

Jan 6, 2017

Merged

Added missing headers in fixture files#21171

nicolas-grekas merged 1 commit intosymfony:3.1fromlinaori:fix/fixture-header

Jan 6, 2017

Conversation

Copy link

Contributor

linaori commentedJan 6, 2017

Q	A
Branch?	3.1
Bug fix?	no
New feature?	no
BC breaks?	no
Deprecations?	no
Tests pass?	n/a
Fixed tickets	~
License	MIT
Doc PR	~

The headers were missing in a few files as mentioned in#21164 (comment)

Added missing headers in fixture files

c9c2474

carsonbot added the Status: Needs Review label

Jan 6, 2017

Copy link

Member

nicolas-grekas commentedJan 6, 2017

Thank you@iltar.

nicolas-grekas merged commitc9c2474 intosymfony:3.1

Jan 6, 2017

nicolas-grekas added a commit that referenced this pull request

Jan 6, 2017

minor#21171Added missing headers in fixture files (iltar)

d7928ea

This PR was merged into the 3.1 branch.Discussion----------Added missing headers in fixture files| Q             | A| ------------- | ---| Branch?       | 3.1| Bug fix?      | no| New feature?  | no| BC breaks?    | no| Deprecations? | no| Tests pass?   | n/a| Fixed tickets | ~| License       | MIT| Doc PR        | ~The headers were missing in a few files as mentioned in#21164 (comment)Commits-------c9c2474 Added missing headers in fixture files