Conversation
danthe1st
left a comment
There was a problem hiding this comment.
Thank you very much for your article and your changes to it. I think it turned out great.
nipafx
left a comment
There was a problem hiding this comment.
Hi Cay. 👋🏾 Looks good but there are a few small fixes to make. I proposed many of them below.
|
@cayhorstmann sorry for the delay, let's get this merged! What does everyone think about making this article the 5th in the I/O series [1]? |
|
I think it might be weird to have that article after Also, I just realized that the file here is just called |
|
I've already fixed the metadata locally. Just need a place to put the tutorial. We are leaning away from the series now. |
|
ok we're considering some possible content reorg strategies but for now I'm thinking keep it where it's at: "Mastering the API" section as its own article, not as part of the I/O series. |
|
ok metadata added, ready for merge here cc @cayhorstmann @ammbra @JosePaumard. |
carimura
left a comment
There was a problem hiding this comment.
LGTM -- looking for one more review from Java team on organization
Should there be a table of contents? |
|
TOC should automatically build itself when missing |
|
boom thanks all! |
|
Thanks for finishing this, guys! 👍🏾 |
No description provided.