Skip to content

Upgrade to PHP 8.1 and using PHPStan#5

Merged
skywarth merged 2 commits intomasterfrom
php-version-upgrade
May 22, 2025
Merged

Upgrade to PHP 8.1 and using PHPStan#5
skywarth merged 2 commits intomasterfrom
php-version-upgrade

Conversation

@skywarth
Copy link
Owner

@skywarth skywarth commented May 19, 2025

Initial upgrade to PHP 8.1 requisite, installing phpstan and fixing static analysis errors. Couldn't upgrade the testbench and phpunit yet, as they failed on the most recent versions.

Upgrade to PHP 8.3 and latter versions will be handled separately, though this should fix all the deprecation notices in #4 regardless.

Scheduler went under massive changes, need to tackle the following TODOs first:

  • Update the documentation in master branch to add warning about later Laravel versions
  • Update the documentation according to the new syntax. Beware that interface API has changed and CRON now allows for secondly runs. (Will be tackled on Laravel 12.x upgrade)
  • Update the tests (Will be tackled on Laravel 12.x upgrade)
  • Manually test in a sample laravel application

…tatic analysis errors. Couldn't upgrade the testbench and phpunit yet, as they failed on the most recent versions
@darkest-pr
Copy link

darkest-pr bot commented May 19, 2025

📜 You will endure this loss, and learn from it.

@skywarth
Copy link
Owner Author

📜 You will endure this loss, and learn from it.
Will I tho? @darkest-PR, what do you think of your chances of making it through in time?

@darkest-pr
Copy link

darkest-pr bot commented May 19, 2025

Replying to: @skywarth

Original message

📜 You will endure this loss, and learn from it.
Will I tho? @darkest-PR, what do you think of your chances of making it through in time?


🕯️ Elusive, evasive, persistent. Righteous traits for a rogue.

@codecov-commenter
Copy link

codecov-commenter commented May 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.16%. Comparing base (e027ccf) to head (7aca8b8).
Report is 4 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master       #5      +/-   ##
============================================
+ Coverage     96.13%   96.16%   +0.03%     
- Complexity      104      107       +3     
============================================
  Files            12       13       +1     
  Lines           336      339       +3     
============================================
+ Hits            323      326       +3     
  Misses           13       13              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@skywarth skywarth mentioned this pull request May 20, 2025
@sonarqubecloud
Copy link

@skywarth skywarth merged commit 9306efd into master May 22, 2025
3 of 4 checks passed
@darkest-pr
Copy link

darkest-pr bot commented May 22, 2025

🕯️ Wealth beyond measure, awarded to the brave and the foolhardy alike.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants