Skip to content

pre-release/v2.0.2 #5

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Aug 27, 2022
Merged

pre-release/v2.0.2 #5

merged 5 commits into from
Aug 27, 2022

Conversation

josantonius
Copy link
Owner

No description provided.

The CamelCaseMethodName rule was disabled to avoid alerts on function names in tests.

Closes #5
Functions were added to document the methods and avoid confusion.

Closes #5
The alignment of the asterisks in the comments has been fixed.

Closes #5
@codecov
Copy link

codecov bot commented Aug 27, 2022

Codecov Report

Merging #5 (6beb865) into main (4defb0b) will not change coverage.
The diff coverage is n/a.

❗ Current head 6beb865 differs from pull request most recent head d2d6169. Consider uploading reports for the commit d2d6169 to get more accurate results

Impacted file tree graph

@@             Coverage Diff             @@
##                main        #5   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity        27        27           
===========================================
  Files              4         4           
  Lines             59        59           
===========================================
  Hits              59        59           
Impacted Files Coverage Δ
src/ErrorException.php 100.00% <ø> (ø)
src/ErrorHandled.php 100.00% <ø> (ø)
src/ErrorHandler.php 100.00% <ø> (ø)
src/Exceptions/WrongErrorLevelException.php 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4defb0b...d2d6169. Read the comment docs.

The notation type in the test function names has been changed from camel to snake case for readability.

Closes #5
@josantonius josantonius merged commit fc9a970 into main Aug 27, 2022
@josantonius josantonius deleted the pre-release/v2.0.2 branch August 27, 2022 20:27
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.

1 participant