ampbench
ampbench copied to clipboard
ampproject
Adding Bing Ads + branch.io icon
Add Bings Ads detection rules + branch.io icon + version bump.
This also fixes LICENSE issue (see https://github.com/ampproject/amphtml/issues/16272)
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. In order to pass this check, please resolve this problem and have the pull request author add another comment and the bot will run again. If the bot doesn't comment, it means it doesn't think anything has changed.
Didn't know to set the correct author email so CLA check broke... what's the best way to change author from "Phillip Kriegel <[email protected]>" to "Phillip Kriegel <[email protected]>"?
@philkrie So, after some git research and experimentation, I can't find any approach better than essentially re-doing this pull request… @jeffjose do you mind if your authorship gets lost? If not, then @philkrie can you squash and force push (or create a fresh PR with the same content)? To preserve authorship I think you'll need to do the same commits again, in the same order.
(I'm sure there is some way to fix this properly, but I don't know it. I did find https://help.github.com/articles/changing-author-info/ but that seems to create (some) changes to the repo even when no email addresses need to change, so I don't really trust it…)
Sounds good, I've gone ahead and fixed the issue in my fork, and submit a PR to @jeffjose 's fork again.
So there's good news and bad news.
:thumbsup: 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.
:confused: 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 cla/google commit status will not change from this state. It's up to you to confirm consent of all the commit author(s), set the cla label to yes (if enabled on your project), and then merge this pull request when appropriate.
The commits are from authors who have already participated in this repo. We should be good to go here. Let me know if that is not the case.
If I understand correctly we just need review and for the cla: yes label to be set to get around googlebot's warning.
@ithinkihaveacat do you know how to proceed from here? As I see it, there's 2 issues. Google CLA and some merge conflicts. The latter should be pretty straightforward, but not sure what to do with the former.