Skip to content

dep-check: USE CAPS FOR IMPORTANCE#26993

Merged
samouri merged 1 commit intoampproject:masterfrom
samouri:dep-check-followup
Feb 27, 2020
Merged

dep-check: USE CAPS FOR IMPORTANCE#26993
samouri merged 1 commit intoampproject:masterfrom
samouri:dep-check-followup

Conversation

@samouri
Copy link
Copy Markdown
Member

@samouri samouri commented Feb 26, 2020

followup to #26981

@samouri samouri requested a review from rsimha February 26, 2020 22:05
@samouri samouri self-assigned this Feb 26, 2020
Copy link
Copy Markdown
Contributor

@rsimha rsimha left a comment

Choose a reason for hiding this comment

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

THANK YOU.

@samouri samouri merged commit 31fbacd into ampproject:master Feb 27, 2020
@mdmower
Copy link
Copy Markdown
Contributor

mdmower commented Feb 27, 2020

So, in all seriousness, wasn't allowlist correct? allowlist is the name of the array that is off limits.

@samouri samouri deleted the dep-check-followup branch February 27, 2020 17:36
@samouri
Copy link
Copy Markdown
Member Author

samouri commented Feb 27, 2020

So, in all seriousness, wasn't allowlist correct? allowlist is the name of the array that is off limits.

Correct. It used to be whitelist / DO NOT ADD TO WHITELIST, so I'm guessing @rsimha just wanted to match how it was prior. I also think it reads better since we are referring to the concept of an allowlist just as much as the specific variable name.

In truth, I'm fine with it uppercase or lowercase. 🤷‍♂

robinvanopstal added a commit to jungvonmatt/amphtml that referenced this pull request Mar 2, 2020
* master:
  Launch `amp-next-page` v2 & clean up experiment (ampproject#27035)
  ✨Implement Display Locking on amp-accordion (ampproject#25867)
  📖<amp-next-page> documentation and validation (ampproject#26841)
  Improve visibility event tests (ampproject#26847)
  amp-geo: Fall back to API for country (ampproject#26407)
  ✨ Add customization options to <amp-story-quiz> (ampproject#26714)
  navigation: Minor test improvements (ampproject#26926)
  ♻️ Alias video.fullscreen action for symmetry with interface names (ampproject#27017)
  ✨ Implements `amp-analytics` support for `amp-next-page` (ampproject#26451)
  ✨ amp-video-iframe integration: global jwplayer() singleton by default (ampproject#26969)
  Fix unit tests broken by ampproject#26687 (ampproject#27000)
  Filter redirect info from emails (ampproject#27012)
  📖 Make amp-bind doc valid, fix amp-form stories filter (ampproject#27003)
  url-replacements: Minor test improvement (ampproject#26930)
  viewport: Minor test improvement (ampproject#26931)
  amp-consent fullscreen warning (ampproject#26467)
  dep-check: USE CAPS FOR IMPORTANCE (ampproject#26993)
  fix img url (ampproject#26987)

# Conflicts:
#	extensions/amp-next-page/amp-next-page.md
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.

5 participants