Skip to content

Fix: Stop invoking useless function - #66

Merged
cmb69 merged 1 commit into
php:masterfrom
localheinz:fix/manual-header
Jun 30, 2022
Merged

Fix: Stop invoking useless function#66
cmb69 merged 1 commit into
php:masterfrom
localheinz:fix/manual-header

Conversation

@localheinz

Copy link
Copy Markdown
Contributor

This pull request

  • stops invoking a useless function manual_header()

💁‍♂️ For reference, see php/web-php#584.

@kamil-tekiela

Copy link
Copy Markdown
Member

I am not convinced. Can we not decouple PHD from WEB? Why is one calling functions from the other?

@localheinz

Copy link
Copy Markdown
ContributorAuthor

@kamil-tekiela

I am not convinced. Can we not decouple PHD from WEB? Why is one calling functions from the other?

I agree, if @cmb69 hadn't caught it - this is an entire mess!

@cmb69

Copy link
Copy Markdown
Member

PhD doesn't call that function, but rather emits the function call to the manual pages.

@philip, do you have any concerns about this change; AFAIK, you're no longer using the latest versions of PhD, but I want to make that we're not breaking anything for you.

@philip

Copy link
Copy Markdown
Member

Thank you @cmb69 for remembering and checking in, I truly appreciate it.

We recently stopped importing PHP's MySQL connector documentation into the MySQL documentation so our use case can be ignored; it's probably safe to treat PhD as a php.net specific project.

@salathe
salathe self-requested a review June 29, 2022 17:31
@cmb69

Copy link
Copy Markdown
Member

Thanks for the confirmation, @philip! And thank you for the PR, @localheinz!

@cmb69
cmb69 merged commit f91df3c into php:masterJun 30, 2022
@localheinz
localheinz deleted the fix/manual-header branch June 30, 2022 09:48
@localheinz

Copy link
Copy Markdown
ContributorAuthor

Thank you, @cmb69, @kamil-tekiela, @philip, and @salathe!

Sign up for freeto 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.

5 participants

@localheinz@kamil-tekiela@cmb69@philip@salathe