Add jsonSerialize as valid method name

[jtgoltz]

Fixing issue: #117

[jtgoltz]

I don't believe that Travis failed because of my code changes. Is there a known issue about some jobs failing?

[stronk7]

I don't believe that Travis failed because of my code changes. Is there a known issue about some jobs failing?

Don't worry, that's Travis CI hitting the new DockerHub pull limits. I've relaunched the failed job.

Soon, we'll be adding support for dockerhub login, it's really simple, see moodlehq/moodle-docker#150

And also, the new GitHub Actions are being added to this repo, see #131. And GHA jobs aren't exposed to those DockerHub limits (they settled an agreement).

Ciao :-)

[stronk7]

Tiny detail... I'd add the missing trailing comma "," at the end of the new line, like the original one had. It's a tiny details but helps to get simpler diffs when there are changes in many lines.

[jtgoltz]

Tiny detail... I'd add the missing trailing comma "," at the end of the new line, like the original one had. It's a tiny details but helps to get simpler diffs when there are changes in many lines.

Such a tiny commit and there is still an error ;)

I pushed a new version

[stronk7]

All ok, approving this, thanks @jtgoltz !