@mjcaparas
Edit complete. A few comments:
- TOC title should be sentence case "Static proxy configuration" instead of title case.
- Line 48: Is "prepended" the intended term?
- Line 73: Implies that two commands follow, but there's only one command beneath.
Thanks!
Kelly
@mjcaparas
Edit complete! Again, we're using "install" and "deploy" interchangeably. In most cases, I think "install" is more accurate and seems to be what's used in the code, but "deploy" has a slightly different meaning and might be more accurate in some instances, so it would be good to get crisper with the usage.
TOC titles don't need to map verbatim to H1 headings, but if you stick with the TOC version, use "Puppet-based installation" (or Puppet-based deployment if switching to that language throughout).
On line 172: do you mean "2 if the device is not yet onboarded"?
Thanks!
Kelly
@mjcaparas
The edit is complete. Since "install" and "deploy" were used interchangeably here, I changed most instances to "install" for consistency since that seemed more accurate. Feel free to make a different call. Note also that I changed the instances of "Microsoft Linux software repository" to just "Linux software repository," as including Microsoft could be problematic from a copyright standpoint. I couldn't find that usage anywhere either, but if you know for sure that Microsoft should be included, you can reject those changes.
Please review and once approved, we can merge.
Thanks,
Kelly
@mjcaparas
Edit is complete. Please review to ensure I didn't change the technical meaning. Once changes are approved, sign off and we can merge.
Thanks!
Kelly
