Skip to content

Addresses comment left over from PR#3841#3853

Merged
lannka merged 1 commit intoampproject:masterfrom
lannka:promise_nit
Jun 30, 2016
Merged

Addresses comment left over from PR#3841#3853
lannka merged 1 commit intoampproject:masterfrom
lannka:promise_nit

Conversation

@lannka
Copy link
Copy Markdown
Contributor

@lannka lannka commented Jun 30, 2016

No description provided.

@jridgewell
Copy link
Copy Markdown
Contributor

LGTM.

@lannka lannka merged commit 0f6dc4b into ampproject:master Jun 30, 2016
@lannka lannka deleted the promise_nit branch June 30, 2016 21:50
@erwinmombay
Copy link
Copy Markdown
Member

@lannka do you mind updating the header of this PR, when we generate the changelog this isn't going to be very helpful from a glance what this PR did

jonasmattsson1 added a commit to widespace-os/amphtml that referenced this pull request Jul 5, 2016
* master: (236 commits)
  trim all the columns (ampproject#3894)
  Refactoring: Turn private custom element methods into functions. (ampproject#3882)
  Lower the load priority of ad shaped iframes. (ampproject#3863)
  JsDoc fix (ampproject#3892)
  Add screenshots for Opera to AMP Validator extension. (ampproject#3866)
  Fix renaming of generated JSCompiler_prototypeAlias variable. (ampproject#3887)
  fix typo in amp-sidebar.md (ampproject#3833)
  Validator Roll-up (ampproject#3885)
  [CryptoService] Leverage browser native Crypto API to hash strings. (ampproject#3850)
  Size update (ampproject#3883)
  copy amp-ad docs to builtins (ampproject#3879)
  move doc to extension (ampproject#3878)
  [amp-experiment] Exposes isDismissed() method in AmpUserNotification (ampproject#3832)
  fix action-impl warning on dist (ampproject#3867)
  Add params for microad (ampproject#3827)
  Fixed some A4A tests. (ampproject#3859)
  Updates to colanalytics vendor config for amp-analytics. (ampproject#3849)
  Changes to implement A4A (AMP ads for AMP pages) (ampproject#3534)
  Addresses comment left over from PR#3841 (ampproject#3853)
  Expose submit event with on=submit:el.action syntax. (ampproject#3739)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants