Closed bmfmancini closed 11 months ago
Is this ready to merge?
No, still outstanding indentation issues :)
@bmfmancini, can you work on the outstanding comments. @netniV, if they are resolved, can you mark such so that we can merge?
Netniv was going to look at this one from what I remember differences between the MD standards and our doc standards
On Sat, Jun 3, 2023, 11:45 AM TheWitness @.***> wrote:
@bmfmancini https://github.com/bmfmancini, can you work on the outstanding comments. @netniV https://github.com/netniV, if they are resolved, can you mark such so that we can merge?
— Reply to this email directly, view it on GitHub https://github.com/Cacti/documentation/pull/171#issuecomment-1575030806, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADGEXTBBSGV7FDHRNN2A24LXJNL2TANCNFSM6AAAAAATTPFKKQ . You are receiving this because you were mentioned.Message ID: @.***>
Yeah I was trying to find a way to make it easier for @bmfmancini to comply with our normal doc standards using an editor such as VSC. i'll try and get back on that this week.
Where is this at?
Checkout Pull 177, which has seans changes, plus my improvements and should close this one when merged.
@netniV I have updated the spacing