Skip to content

Add license to the Getting Started guides#7663

Merged
camelburrito merged 2 commits intoampproject:masterfrom
mrjoro:add-license
Feb 18, 2017
Merged

Add license to the Getting Started guides#7663
camelburrito merged 2 commits intoampproject:masterfrom
mrjoro:add-license

Conversation

@mrjoro
Copy link
Copy Markdown
Member

@mrjoro mrjoro commented Feb 18, 2017

I forgot to include licenses in the Getting Started guides on the initial submit.

cc @cramforce

@mrjoro mrjoro added this to the Docs Updates milestone Feb 18, 2017
@mrjoro mrjoro self-assigned this Feb 18, 2017
@mrjoro mrjoro requested a review from camelburrito February 18, 2017 20:01
@camelburrito camelburrito merged commit 7ce017c into ampproject:master Feb 18, 2017
@mrjoro mrjoro deleted the add-license branch February 19, 2017 02:36
@@ -1,3 +1,19 @@
<!---
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just a note: We are inconsistent (because it makes little sense), but our rules do not require this header on MD files. For checked in code a presubmit enforces that files are properly annotated with a license reference.

mrjoro added a commit to mrjoro/amphtml that referenced this pull request Apr 28, 2017
* adding license

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants