Closed emmurphy1 closed 3 years ago
@mikemckiernan and @ncbaratta I have updated with your comments, or explained. Please review when you have time.
@rkratky @Preeticp @ncbaratta I've revised the text fragments section. Would you mind looking over again and approving if you can?
@rkratky @ncbaratta @rolfedh @fbolton @fbolton @sterobin @Preeticp @mikemckiernan @yzimmerm I think I have addressed everyone's comments but there have been a few changes so would you mind looking over one last time? Also, I am now second-guessing myself and wondering if we should be talking about 'snippets' instead of 'fragments'? Ideas?
It still lgtm. My preference for fragments over snippets is to sidestep possible confusion between a text file with limited text content and the feature in Atom that is named snippets and inserts small pieces of text.
I've been saying 'snippets' but seriously I have no preference.
This continues the work done in https://github.com/redhat-documentation/modular-docs/pull/123/files. It resolves issue 116 and issue 99.