Skip to content
This repository was archived by the owner on Jan 29, 2020. It is now read-only.

Fix license header #353

Merged

Conversation

malukenho
Copy link
Member

If the tool to check license headers is considered overkill, just tell me, so I can remove it.

@geerteltink
Copy link
Member

Considering all the errors found I don't think it's overkill :)

@malukenho malukenho closed this Jun 7, 2016
@malukenho malukenho reopened this Jun 7, 2016
@@ -31,7 +31,8 @@
"zendframework/zend-expressive-aurarouter": "^1.0",
"zendframework/zend-expressive-fastroute": "^1.0",
"zendframework/zend-expressive-zendrouter": "^1.0",
"zendframework/zend-servicemanager": "^2.6"
"zendframework/zend-servicemanager": "^2.6",
"malukenho/docheader": "^0.1.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

should be sorted alphabetically ( before phpunit list )

@weierophinney weierophinney self-assigned this Nov 10, 2016
@weierophinney weierophinney merged commit 310e1d0 into zendframework:master Nov 10, 2016
weierophinney added a commit that referenced this pull request Nov 10, 2016
weierophinney added a commit that referenced this pull request Nov 10, 2016
weierophinney added a commit that referenced this pull request Nov 10, 2016
Forward port #353

Conflicts:
	composer.json
@weierophinney
Copy link
Member

Thanks, @malukenho. I modified the docheader file to reflect the style we use, which omits the initial header lines, and then updated all files flagged.

@malukenho malukenho deleted the enhancement/fix-license-header branch November 11, 2016 01:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants