Skip to content

Added reference to common attributes + minor improvements (has dependency PR)#6772

Merged
1 commit merged intomasterfrom
unknown repository
Dec 22, 2016
Merged

Added reference to common attributes + minor improvements (has dependency PR)#6772
1 commit merged intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Dec 21, 2016

Related to ampproject/amp.dev#276 - adding documentation for common attributes.

to: @pbakaus
cc: @cramforce

Note: Do not merge until ampproject/amp.dev#276 is in production.

Copy link
Copy Markdown
Contributor

@pbakaus pbakaus left a comment

Choose a reason for hiding this comment

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

LGTM.

@ghost ghost self-assigned this Dec 21, 2016
@ghost
Copy link
Copy Markdown
Author

ghost commented Dec 21, 2016

Barb will merge when other PR is in production

@jridgewell
Copy link
Copy Markdown
Contributor

@bpaduch: Add a review "requesting changes". Then you'll have to approve before this is merged.

@ghost
Copy link
Copy Markdown
Author

ghost commented Dec 21, 2016

@jridgewell - How is that done if I'm the owner of the PR?

@jridgewell
Copy link
Copy Markdown
Contributor

Go to "Files Changed" at the top, Click the "Review Changes" button, then select "Request changes" and submit.

@ghost ghost merged commit 0c051a2 into ampproject:master Dec 22, 2016
@ghost ghost deleted the docs-attributes-ampimg branch December 22, 2016 16:55
torch2424 pushed a commit to torch2424/amphtml that referenced this pull request Jan 3, 2017
jridgewell pushed a commit to jridgewell/amphtml that referenced this pull request Jan 31, 2017
This pull request was closed.
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.

2 participants