-
Couldn't load subscription status.
- Fork 39
Add Bing Ads + branch.io icon (redo) #58
Conversation
…i (needs more testing), Wordpress (Wappalyzer regex), and Adobe Experience Manager (Wappalyzer regex). Still uses legacy Wappalyzer JSON formatting.
|
So there's good news and bad news. 👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there. 😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request. Note to project maintainer: This is a terminal state, meaning the |
|
A Googler has manually verified that the CLAs look good. (Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.) |
|
@googlebot CLAs are approved for commit authors @jeffjose ([email protected]) and @philkrie ([email protected]); see discussion in #46. @jeffjose and @philkire: can you confirm that this PR is what you are expecting? |
|
LGTM. Please wait for @philkrie to confirm his changes before merging. Thanks! |
|
Looks good to me. @jeffjose: Should we merge now or wait to add the full update to use the latest Wappalyzer JSON? |
|
This PR is now obsolete. We moved readiness-tool from this repo onto ampproject/amp-readiness. Please feel free to close this PR out WITHOUT merging. |
This has the same changes as #46, just rebased on top of master.
This will hopefully: (a) make the diff cleaner; (b) resolve conflicts; and (b) shut googlebot up.