_user_mail_notify -- options for $op need to be a list

Thanks for reporting this! Good novice project, and needs to be done in Drupal 8, then 7, then 6.

Patch adds correct indentation and "-" for the possible values for function _user_mail_notify() description. Patch is against 8.x

Looks close, thanks!

Please check out our list guidelines though -- every list item should end with a . :
http://drupal.org/node/1354#lists
You might also verify that none of those lines exceeds 80 characters, just in case.

Also, we need a blank line between @param and @return.

Thanks for catching and fixing the indentation on the param/returns too!

Thank you for your guideance.
New patch adds "." to end of list items and a blank line between @param and @return.
Regarding line lengths, does this count as 80 or 81?

 *   - 'register_no_approval_required': Welcome message when user self-registers.

New patch ensures line length of 80 characters.

Looks good, thanks!

Looks great!

Committed and pushed to 8.x and 7.x. Thanks!

Still needs backport to D6.

Attached patch is against 6.x branch. Changed the indentation, added "-" to seperate list items, added punctuation and blank row before return value.

Thanks! It looks like there is one other mistake to correct in 6.x -- there are two spaces instead of one after the . in this line:

+ *   The user object of the account being notified.  Must contain at

New patch fixes the two spaces noticed in #11.

Looks good, thanks!

Thanks a lot, committed, pushed.