Skip to content

Comments

Small optimization of getting name of the JSON error constant#11

Open
peter-gribanov wants to merge 1 commit intoviolet-php:mainfrom
peter-gribanov:json_error_constant_name
Open

Small optimization of getting name of the JSON error constant#11
peter-gribanov wants to merge 1 commit intoviolet-php:mainfrom
peter-gribanov:json_error_constant_name

Conversation

@peter-gribanov
Copy link

We restrict the selection to only JSON category constants so as not to check all constants.

@codecov
Copy link

codecov bot commented Jan 27, 2023

Codecov Report

Merging #11 (c72dd5c) into main (4de0ac8) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files
Impacted Files Coverage Δ
src/AbstractJsonEncoder.php 100.00% <100.00%> (ø)
src/JsonStream.php 100.00% <0.00%> (ø)

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