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

docs(ngShowHide): improve consistency in ngShowHide documentation #9081

Closed
wants to merge 1 commit into from

Conversation

maip
Copy link
Contributor

@maip maip commented Sep 14, 2014

add backticks around directive names to improve documentation consistency

@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.

@caitp
Copy link
Contributor

caitp commented Sep 14, 2014

There are a few instances of class names used in the document that didn't get backtick'd --- do you want to add those too? (the animation subheading in particular)

add backticks around directive names to improve documentation consistency
@maip
Copy link
Contributor Author

maip commented Sep 15, 2014

I added some more backticks and I submitted another CLA with an updated email.

@caitp
Copy link
Contributor

caitp commented Sep 15, 2014

Don't worry about the CLA signature (it's not necessary for docs), however it needs to be signed with nokami.hybrids@... for this patch (because that's the address this was authored with).

@maip
Copy link
Contributor Author

maip commented Sep 15, 2014

Oh ok, do I update the commit with my email then? How do I sign it?

@caitp
Copy link
Contributor

caitp commented Sep 15, 2014

you just sign through the form (instructions in CONTRIBUTING.md, linked in mp's first message) --- it's just that for mp to recognize that you've signed it, you need to sign with the email address that you authored your patch with (which frequently confuses people, myself included)

@caitp caitp closed this in ea94e63 Sep 15, 2014
@caitp
Copy link
Contributor

caitp commented Sep 15, 2014

The patch doesn't apply cleanly for v1.2.x, if you want to sort out the conflicts and submit a second pull request there we could merge it there too. Thanks for the patch.

petebacondarwin added a commit to petebacondarwin/angular.js that referenced this pull request Sep 15, 2014
…ames

add backticks around directive names to improve documentation consistency, as it's used in some
parts of the docs already.

Closes angular#9081
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.

3 participants