Skip to content

Update docblocks with official documentation #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 30, 2018
Merged

Update docblocks with official documentation #4

merged 1 commit into from
Oct 30, 2018

Conversation

shadowhand
Copy link
Collaborator

@shadowhand shadowhand commented Oct 26, 2018

Interface documentation was not updated after PSR-15 was approved, which makes these interfaces out of sync with the accepted specification.

Refs php-fig/fig-standards#1107

@shadowhand
Copy link
Collaborator Author

If approved, this should be tagged as version 1.0.1.

Copy link
Member

@Jean85 Jean85 left a comment

Choose a reason for hiding this comment

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

Here too, LGTM 👍 in reference to php-fig/fig-standards#1107 (comment)

@Jean85
Copy link
Member

Jean85 commented Oct 30, 2018

Ping me if you do not have permissions for merge/release.

@shadowhand
Copy link
Collaborator Author

I'm not sure who has ownership over these repos. Maybe @weierophinney ?

@Jean85
Copy link
Member

Jean85 commented Oct 30, 2018

As editor yes, he has access.

@weierophinney
Copy link

Um, @shadowhand is editor; I'm sponsor. 😄

I don't have write access to this repo, based on what GitHub is reporting to me currently...

@Jean85
Copy link
Member

Jean85 commented Oct 30, 2018

Woops sorry, I mixed those up.

Well, the editor should retain access to the repos, even after approval; hence, I've given write access to @shadowhand

@shadowhand shadowhand merged commit aff2f80 into php-fig:master Oct 30, 2018
@shadowhand shadowhand deleted the fix/phpdocs branch October 30, 2018 16:46
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