Update pimple container version

[NikhilJoshua]

Summary

• Update Pimple version to 3.5.* in composer.json for PHP 8.1 Compatibility

closes #161

[github-actions]

Code coverage report: https://gh-reports.rt.gw/login-with-google/refs/pull/162/merge/

[pavanpatil1]

Hi @NikhilJoshua, Verified the PR but while activating the plugin there's fatal error is visible. Please check:

[NikhilJoshua]

@pavanpatil1 Can you please try deleting the vendor folder and do a composer install ?

[pavanpatil1]

Hi @NikhilJoshua, after executing the above steps able to activate the plugin successfully. However, user should not perform this extra step to activate the plugin. Could you please check and update the require dependencies.

[NikhilJoshua]

@pavanpatil1 I believe the issue was specific to your system, was not able to replicate in local. Can you please check in any other system to confirm it's not an issue from your side.

[pavanpatil1]

@NikhilJoshua that shouldn't be the case as the GH action is also failing on same issue

Still, I will cross-checking this once again on rt.gw site

[github-actions]

Code coverage report: https://gh-reports.rt.gw/login-with-google/refs/pull/162/merge/

[pavanpatil1]

Hi @NikhilJoshua, verified the fix and it is working fine now. Able to activate the plugin successfully and it is working fine. However, there are still some deprecated notices after adding the login with google block -

[rtBot]

Code analysis identified issues

action-phpcs-code-review has identified potential problems in this pull request during automated scanning. We recommend reviewing the issues noted and that they are resolved.

phpcs scanning turned up:

🚫 12 errors

⚠️ 4 warnings

---

Powered by rtCamp's GitHub Actions Library

Posting will continue in further review(s)

[rtBot]

Previous scan continued.

[github-actions]

Code coverage report: https://gh-reports.rt.gw/login-with-google/refs/pull/162/merge/

[pavanpatil1]

Verified the fix and it is working fine now. The deprecated notices are not visible now. @NikhilJoshua you can merge the PR to master.

Just the check the inspection check why it it failing.

[github-actions]

Code coverage report: https://gh-reports.rt.gw/login-with-google/refs/pull/162/merge/

[NikhilJoshua]

@pavanpatil1 Fixed the phpcs scan on vendor folder. Please check.

[pavanpatil1]

Hi @NikhilJoshua, It is working fine now. Please get the PR reviewed and merged.

[vishalkakadiya]

LGTM!

vendor folder was not ignored while checking.