Add 2 more tests and many more typehints #128
Merged
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.
Dear @khanamiryan
I want to thank you again for this awesome library and for merging my last PR so fast.
Unfortunately, I had not yet pushed the most important commit, which told composer that the new version is only compatible with PHP 8.1. People noticed (#123, #124, #126) and had some problems.
In this PR, the required line in composer.json is included (as it is also in #125 and #127).
Additionally, I add two more tests and lots of additional typehints, as well as GitHub Actions to run tests and static tests.
The two tests use the image provided in #112, so you could close that one afterwards.
I would suggest that, if you approve my changes, you publish this version as a major one (v2.0.0) and delete version 1.0.6. That would solve the issues people had.
Generally, this PR at least potentially closes #112, #118, #123, #124, #125, #126, #127.
Thanks and best regards,
Tim