Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the Change
This PR goes through our scaffold and adds the correct return types to methods and functions. This will increase the reliability of static analysis available to engineers, either via their IDEs or a tool like PHPStan.
Additionally, it ensures that we, as a company, are starting engineers off on the best foot possible, with no potential errors caused by incorrect return types.
Finally, there are some places where I've added additional defensive coding to ensure that we don't inadvertently trigger errors.
How to test the Change
Test the scaffold locally.
Changelog Entry
Credits
Props @darylldoyle
Checklist:
I have updated the documentation accordingly.I have added Critical Flows, Test Cases, and/or End-to-End Tests to cover my change.