Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Use whitespace to improve readability #9572

Closed
wants to merge 1 commit into from
Closed

Use whitespace to improve readability #9572

wants to merge 1 commit into from

Conversation

LegalizeAdulthood
Copy link
Contributor

No description provided.

@mary-poppins
Copy link

I'm sorry, but I wasn't able to verify your Contributor License Agreement (CLA) signature. CLA signature is required for any code contributions to AngularJS.

Please sign our CLA and ensure that the CLA signature email address and the email address in this PR's commits match.

If you signed the CLA as a corporation, please let us know the company's name.

Thanks a bunch!

PS: If you signed the CLA in the past then most likely the email addresses don't match. Please sign the CLA again or update the email address in the commit of this PR.
PS2: If you are a Googler, please sign the CLA as well to simplify the CLA verification process.

@LegalizeAdulthood
Copy link
Contributor Author

Sorry, I don't give out that level of personal information in order to make contributions to open source. (My address and phone number? Really? Suppose I don't have a phone? I can't contribute? Suppose I have no fixed mailing address?)

@caitp
Copy link
Contributor

caitp commented Oct 13, 2014

you don't need to sign the CLA for changes like this, the robot will not discriminate in asking you to sign because it can't tell what type of fix you're submitting.

caitp pushed a commit that referenced this pull request Oct 13, 2014
☆.。.:・゜☆ Merci (◜௰◝) ☆.。.:・゜☆

Closes #9572
@caitp caitp closed this in a708632 Oct 13, 2014
@LegalizeAdulthood LegalizeAdulthood deleted the spaceman branch October 19, 2014 18:15
@pkozlowski-opensource
Copy link
Member

@LegalizeAdulthood your change was merged as a708632

caitp closed this in a708632 7 days ago

You see different commit sha as we are not merging directly using GitHub's merge button but rather doing an equivalent of cherry-pick.

So yes, your change was merged.

@LegalizeAdulthood
Copy link
Contributor Author

Thanks for explaining it to me. I didn't initially notice that the request was closed with a commit that incorporated the changes. After noticing I deleted my comment.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants