appwrite icon indicating copy to clipboard operation
appwrite copied to clipboard

🔎 Add Code Analyzer To ‘utopia-php/messaging’

Open christyjacob4 opened this issue 3 years ago • 8 comments
trafficstars

💭 Introduction

Code analyzer is one of the most valuable tools in the open source world. It not only ensures every high quality of the code but also allows CI/CD to ensure this. A code analyzer can find a potential bug, unexpected null value, or type mismatch. Having a code analyzer set up should be a high priority for every project as it makes the project more stable and bug-proof.

Your task is to update our library utopia-php/messaging and implement a code analyzer. We would like to use PHPStan as our analyzer.

To make this task easy, we created a sample pull request showing what exactly needs to be done to implement the analyzer into the Utopia PHP library: https://github.com/utopia-php/ab/pull/8

🎯 Requirements

  • Experience with GitHub Actions.
  • Experience with Composer.
  • Experience with PHP.

✅ Task Summary

Please configure PHPStan to max level (9). If that gives you too many problems that you cannot solve, please drop to level 8. If you have trouble with level 8 as well, let us know, and we can consider a lower level on some repositories.

If you have questions, need any help, or just want to hang out, make sure to join us on our Discord server.

Happy Appwriting!

christyjacob4 avatar Oct 10 '22 09:10 christyjacob4

Can I work on this?

maxslimb avatar Oct 10 '22 13:10 maxslimb

@maxslimb , looks like you have some issues assigned to you already. We request you to complete those before requesting assignments on new ones. Also, please consider picking a different type of issue so other contributors can try this one.

Meldiron avatar Oct 11 '22 07:10 Meldiron

Done with the previous Author remove the task. Interested in this issue. Please assign me. Thanks

Ambareen09 avatar Oct 11 '22 16:10 Ambareen09

Hey! Can I work on this issue please?

BabyElias avatar Oct 11 '22 17:10 BabyElias

@Ambareen09 Thanks for your interest! You may work on this issue 🙏 Happy hacking! 🎃

stnguyen90 avatar Oct 11 '22 23:10 stnguyen90

@BabyElias, we're assigning people on a first come first serve basis. If @Ambareen09 isn't able to complete this issue, we'll reassign it accordingly. In the meantime, you can find another Hacktoberfest issue via our website.

stnguyen90 avatar Oct 11 '22 23:10 stnguyen90

I have opened a PR, kindly review it!

Ambareen09 avatar Oct 14 '22 10:10 Ambareen09

@Ambareen09 Thanks for the PR! 🤯 Please give us some time to review it.

stnguyen90 avatar Oct 14 '22 23:10 stnguyen90

Thank you everyone for celebrating Hacktoberfest 22 with us! This issue will now be closed as we're getting ready to celebrate Hacktoberfest 23.

eldadfux avatar Aug 24 '23 09:08 eldadfux