add json support to Pointer

[trim21]

Before opening your pull request, please make sure that you've:

• updated the changelog if the change is user-facing;
• added tests to cover your changes;
• run the test suite locally (make test); and finally,
• run the linters locally (make lint).

close #178

[CLAassistant]

All committers have signed the CLA.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (6187ac0) to head (6303253).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##            master      #180   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           23        23           
  Lines          394       402    +8     
=========================================
+ Hits           394       402    +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[r-hang]

Could a test case be added for the Unmarshal case?

[trim21]

not sure what are you asking, do you mean line 112?

[trim21]

Ok, I get it. you mean Unmarshal error case

[r-hang]

Thank you for your contribution! This change looks good to me! Left a comment about adding a test case for Unmarshal error.

[ccoVeille]

Maybe the typo in PR and first commit could be fixed

Porinter => Pointer

Code and documentation are OK.

[trim21]

Maybe the typo in PR and first commit could be fixed

Porinter => Pointer

Code and documentation are OK.

can this pr be merged with squash？ I fixed pr title but i don't want to use force push to fix commit message

[ccoVeille]

I will let maintainers reply.

But, I don't think force push would be a problem here.

Maybe you can wait for the review to be done. But each repository has its rules/norms/habits

[r-hang]

Hey @trim21 and @ccoVeille, sorry for the delay. We always merge the PR with a "squash and merge". I've updated the message to change the word to pointer.