search

LibreCodeCoop / whatsapp-client

WhatsApp web php client
GNU Affero General Public License v3.0
41 stars 12 forks source link

Feature/add php stan #16

Closed joncojonathan closed 3 years ago

joncojonathan commented 3 years ago

I've added PHPStan composer.json and to scripts and set composer to run the test. Level is set to 8. I'm struggling to work out how to set strict_types = 1 unless it's just part of the PHPStan arguments in composer (still reading the docs for PHPStan).

Hopefully this partially addresses #4 ?

joncojonathan commented 3 years ago

I'll look to resolve those conflicts and update tomorrow.

joncojonathan commented 3 years ago

Hello @vitormattos,

Hopefully that's sorted now?

joncojonathan commented 3 years ago

The only conflict now seems to be the composer.lock hash value:

<<<<<<< feature/addPHPStan
    "content-hash": "0fc50fc4a4ed3f27c9c9e221277936b6",
=======
    "content-hash": "d0b188a5dd26844a21fca9cc63398c58",
>>>>>>> master

My composer.lock should be the latest, I believe?

joncojonathan commented 3 years ago

Hi @vitormattos. Is this now ready to merge? I would think the composer.lock file needs to keep the hash as I've submitted it due to the changes in my file. I can apply the original value from master (which resolves the conflict) but I don't think that would be correct for Composer. Any advice appreciated, thanks in advance.

vitormattos commented 3 years ago

Is possible solve the problem reported in CI?

joncojonathan commented 3 years ago

Hello, the only remaining problems are what PHPStan has found, which are out of scope for this PR. Are you happy to approve and merge?