[1.5.x] Documentation - "Serialization" page grammar improvements.

[BillyAutrey]

Pull Request Checklist

• Have you read through the contributor guidelines?
• Have you signed the Lightbend CLA?
• Have you updated the documentation?

Fixes

N/A

Purpose

Improves grammar and sentence structure in key places on the "Serialization" Scala page. Improves readability, without changing the content.

[ignasi35]

This looks great @BillyAutrey, thanks!

Would it be possible for you to make the same improvements on the /java/guide version of this same file (Lagom has each page in the docs duplicated)

[ignasi35]

Note to self (and maintainers): this PR targets 1.5.x we'll have to forward port (unless @BillyAutrey rebuilds the PR on top of 1.6.x and we let mergify and the status:needs-backport-1.5label do their magic).

[BillyAutrey]

Sure @ignasi35, I can add a commit for the Java docs. I can also look at the 1.6.x docs and add another PR for that. Should be a straightforward comparison.

[octonato]

@BillyAutrey, you don't need to send another PR. Let me try to re-target this one.

[octonato]

Hmm, didn't work. The best is to close it and send it to 1.6.x

[ignasi35]

Hmm, didn't work. The best is to close it and send it to 1.6.x

rebase the branch on top of 1.6.x and force push?

[ignasi35]

rebase the branch on top of 1.6.x and force push?

The contents of the Serialization.md pages has changed a lot between branches 1.5.x so the back/forward-porting causes a good deal of CONFLICTs.

I think we're better off just targeting 1.5.x and, eventually, we'll review the contents on 1.6.x (if amendments are necessary there).

[ignasi35]

Would it be possible for you to make the same improvements on the /java/guide version of this same file (Lagom has each page in the docs duplicated)

I was cmparing the changes in this PR with the contents on the java/guide just to realise the contents of Serialization.md are extremely different in each implementation (because one uses jackson and the other uses play-json).

[ignasi35]

I think this should be merged without forward-porting and without any changes on java/guide.

thanks @BillyAutrey !

[ignasi35]

I think this should be merged without forward-porting and without any changes on java/guide.

thanks @BillyAutrey !