Skip to content

Added documentation for the str_contains#216

Closed
emmanix2002 wants to merge 3 commits intophp:masterfrom
emmanix2002:feature/add-str-contains-doc
Closed

Added documentation for the str_contains#216
emmanix2002 wants to merge 3 commits intophp:masterfrom
emmanix2002:feature/add-str-contains-doc

Conversation

@emmanix2002
Copy link
Copy Markdown

Added documentation for str_contains based on the RFC https://wiki.php.net/rfc/str_contains

Copy link
Copy Markdown
Member

@Girgias Girgias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the .gitignore file as the docs are backed up by SVN

XML indentation is 1 space, PHP examples 4 space indentation

Modifying the See Also sections of other functions to point to this one would be a good idea

@emmanix2002
Copy link
Copy Markdown
Author

Remove the .gitignore file as the docs are backed up by SVN

XML indentation is 1 space, PHP examples 4 space indentation

Modifying the See Also sections of other functions to point to this one would be a good idea

Got it

@cmb69
Copy link
Copy Markdown
Member

cmb69 commented Nov 23, 2020

Please also add a respective entry to https://github.com/php/doc-en/blob/master/reference/strings/versions.xml.

- Added function reference to the see also section of some related functions
- Reduced XML indentation to 1 space char and updated examples
@emmanix2002
Copy link
Copy Markdown
Author

@Girgias @cmb69 I've attended to your review comments

@emmanix2002
Copy link
Copy Markdown
Author

@Girgias @cmb69 I'm waiting for this to be done so I get a hang of the process before I continue with the str_starts_with and str_ends_with functions.

Copy link
Copy Markdown
Member

@Girgias Girgias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than the nits LGTM

@emmanix2002
Copy link
Copy Markdown
Author

Other than the nits LGTM

@Girgias Fixed

@cmb69
Copy link
Copy Markdown
Member

cmb69 commented Nov 23, 2020

Very nice, thanks! Applied as http://svn.php.net/viewvc?view=revision&revision=351548.

@cmb69 cmb69 closed this Nov 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants