Skip to content

Guzzle: Fix deprecation notice in getConfig#491

Merged
VerifiedJoseph merged 1 commit intomainfrom
guzzle-fix-deprecation-notice
Aug 27, 2025
Merged

Guzzle: Fix deprecation notice in getConfig#491
VerifiedJoseph merged 1 commit intomainfrom
guzzle-fix-deprecation-notice

Conversation

@VerifiedJoseph
Copy link
Owner

@VerifiedJoseph VerifiedJoseph commented Aug 27, 2025

Fixes deprecation notice in Guzzle class function getConfig. closes #490

Deprecated: Optional parameter $auth declared before required parameter $handlerStack is implicitly treated as a required parameter in verifiedjoseph/gotify-api-php/src/Guzzle.php on line 195

@VerifiedJoseph VerifiedJoseph self-assigned this Aug 27, 2025
@VerifiedJoseph VerifiedJoseph force-pushed the guzzle-fix-deprecation-notice branch from 6d63aa1 to 0faf329 Compare August 27, 2025 09:54
@codecov
Copy link

codecov bot commented Aug 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (3073f99) to head (0faf329).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##                main      #491   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity        77        77           
===========================================
  Files             15        15           
  Lines            283       283           
===========================================
  Hits             283       283           

☔ 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.

@VerifiedJoseph VerifiedJoseph merged commit 47723ac into main Aug 27, 2025
5 checks passed
@VerifiedJoseph VerifiedJoseph deleted the guzzle-fix-deprecation-notice branch August 27, 2025 11:39
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.

Guzzle getConfig Deprecation notice

1 participant