Skip to content

pre-release/v2.0.2#5

Merged
josantonius merged 5 commits intomainfrom
pre-release/v2.0.2
Aug 27, 2022
Merged

pre-release/v2.0.2#5
josantonius merged 5 commits intomainfrom
pre-release/v2.0.2

Conversation

@josantonius
Copy link
Copy Markdown
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
Copy Markdown

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