Uh oh!
There was an error while loading. Please reload this page.
Added Code Analyser - #28
Conversation
BabyElias
commented
Oct 14, 2022
Made all corrections, reduced level to 6 |
stnguyen90
left a comment
There was a problem hiding this comment.
Seems like there are a lot of syntax errors now.
| - PHP 7.3.x | ||
| - PHP 7.4.x | ||
| - Different version (specify in environment) | ||
| - PHP 8.0 |
There was a problem hiding this comment.
Please leave the "Different version..." option at the end.
| * @throws \Exception | ||
| */ | ||
| public function call(string $method, string $path = '', array $headers = array(), array $params = array()): array|string | ||
| public function call(string $method, string $path = '', array $headers = array(), array $params = array()): array<int|string,mixed>|string |
There was a problem hiding this comment.
Looks like adding the <> stuff isn't supported and is causing some of the test errors.
PineappleIOnic
commented
Oct 31, 2022
Thank you so much for the PR 🤩. We're adding the |
christyjacob4
commented
Apr 6, 2023
@BabyElias thanks a lot for your contributions during Hacktoberfest 2022! Please reach out to me on our Discord server if you would like to claim your Appwrite swags! As a way of saying thank you, we would also love to invite you to join the Appwrite organization on GitHub. Please share your GitHub username with us on Discord. There is still some work pending on this PR and I haven't got a chance to fix all the errors yet. Are you willing to continue this PR ? |
Solves Issue #4376
But, I need help regarding some diagnosed code issues in order to correct them.
