Skip to content

Json: missing @throws JsonException #212

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.

Sign up for GitHub

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

Jump to bottom
Merged
dg merged 1 commit into from
Jan 21, 2020
Merged

Json: missing @throws JsonException #212

dg merged 1 commit into from
Jan 21, 2020

Conversation

@ddebin
Copy link
Contributor

@ddebin ddebin commented Jan 21, 2020

  • bug fix
  • BC break? no

Adds missing PHPDoc @throws JsonException in JSON::encode and JSON::decode.

@dg
Copy link
Member

dg commented Jan 21, 2020

Thanks

@dg dg changed the title Fix missing @throws JsonException Json: missing @throws JsonException Jan 21, 2020
@dg dg merged commit 4452a30 into nette:master Jan 21, 2020
dg pushed a commit that referenced this pull request Jan 22, 2020
@ddebin @dg
Json: missing @throws JsonException (#212)
48d55a5
dg pushed a commit that referenced this pull request Jan 22, 2020
@ddebin @dg
Json: missing @throws JsonException (#212)
81299a3
dg pushed a commit that referenced this pull request Jan 24, 2020
@ddebin @dg
Json: missing @throws JsonException (#212)
f0d0f19
dg pushed a commit that referenced this pull request Feb 9, 2020
@ddebin @dg
Json: missing @throws JsonException (#212)
976f477
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ddebin @dg