Skip to content

add missing guzzle-exception annotations#1055

Merged
ramsey merged 1 commit intothephpleague:masterfrom
asteragos:add-missing-guzzle-exception-annotations
Feb 26, 2025
Merged

add missing guzzle-exception annotations#1055
ramsey merged 1 commit intothephpleague:masterfrom
asteragos:add-missing-guzzle-exception-annotations

Conversation

@asteragos
Copy link
Contributor

This PR enhances exception documentation by adding missing throws-annotations for guzzle exceptions like ConnectException, which can be thrown across the package. It builds upon the changes made in #1050.

Changes
Added throws-annotations for guzzle exceptions in method doc blocks.

@codecov
Copy link

codecov bot commented Jan 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (7a4e44d) to head (c50e55e).
Report is 3 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##              master     #1055   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity       193       193           
===========================================
  Files             20        20           
  Lines            521       521           
===========================================
  Hits             521       521           
Files with missing lines Coverage Δ
src/Provider/AbstractProvider.php 100.00% <ø> (ø)

@ramsey
Copy link
Contributor

ramsey commented Feb 26, 2025

Thank you for contributing! 🎉

@ramsey ramsey merged commit 56091ce into thephpleague:master Feb 26, 2025
26 checks passed
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