Skip to content

doc about property initialization#7593

Merged
orklah merged 2 commits intovimeo:masterfrom
orklah:docproperties
Feb 5, 2022
Merged

doc about property initialization#7593
orklah merged 2 commits intovimeo:masterfrom
orklah:docproperties

Conversation

@orklah
Copy link
Copy Markdown
Collaborator

@orklah orklah commented Feb 5, 2022

This aims to improve documentation about issues related to property initialization. English is not my first language so feedback is welcome here :)

@orklah orklah added the release:docs The PR will be included in 'Docs' section of the release notes label Feb 5, 2022
Comment thread docs/running_psalm/issues/MissingConstructor.md
Comment thread docs/running_psalm/issues/MissingConstructor.md
@AndrolGenhald
Copy link
Copy Markdown
Collaborator

AndrolGenhald commented Feb 5, 2022

I assume it's possible for non-maintainers to use that suggested changes thing and I just don't know how? I'm having trouble finding documentation on that.

Edit: nvm, figured it out.

Comment thread docs/running_psalm/issues/PropertyNotSetInConstructor.md Outdated
Comment thread docs/running_psalm/issues/PropertyNotSetInConstructor.md Outdated
Comment thread docs/running_psalm/issues/RedundantPropertyInitializationCheck.md Outdated
Comment thread docs/running_psalm/issues/RedundantPropertyInitializationCheck.md Outdated
Co-authored-by: AndrolGenhald <AndrolGenhald@users.noreply.github.com>
@orklah
Copy link
Copy Markdown
Collaborator Author

orklah commented Feb 5, 2022

Thanks for helping :)

@orklah orklah merged commit 3a2b412 into vimeo:master Feb 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:docs The PR will be included in 'Docs' section of the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants