Thiru
Thiru
My (first attempt) implementation of the Text Fragment feature - i had the audacity to include a new flag "include-text-fragments" to explicitly support the fragment directive (aka text fragment) feature....
I have fixed few lint and clippy issues (missed it in the first couple of commits :-() - do review at your available time and let me know your comments!
I missed to run the clippy across the test modules - the related lint failure issues are now fixed and ready for review!
> Left some comments. I like the overall structure. Good work so far! @mre I'll take a look at each of the comments and work to address it - thank...
> @thiru, any updates? Let me know in case you need any help. 😃 @mre, I will submit the fixes by this weekend for your review. Earlier I started moving...
> Thanks, sounds good! @mre my apologies for the delay - while refactoring, into a separate crate, I encountered few corner case issues and it took more time than planned...
I am committing the changes into my fork and will initiate a pull-request shortly!
@mre request your help in addressing the CI / publish-check failure - please recommend if I've to make any changes on my end - thank you!
> I've added a few more comments. Most of them are minor. I think it should be fairly easy to go through them and accept/reject my suggestions from the GitHub...
> @thiru-appitap, I saw that you did some work. Can you close the resolved conversations already? It becomes a bit hard to keep track of the open TODOs. 😉 Also,...