[2.x] Includes facade improvements

[emmadesilva]

• Methods in the Includes facade now return HtmlString objects instead of string
  • Fixes RFC: Should Includes helpers returning Html return HtmlStrings? #1734
• Markdown includes are now converted to HTML using the custom HydePHP Markdown service, meaning they now support full GFM spec and custom Hyde features like colored blockquotes and code block filepath labels
• Markdown returned from includes are now trimmed of trailing whitespace and newlines
• The Includes::path() and Includes::get() methods now normalizes paths to be basenames to match the behaviour of the other include method.
  • This means that nested directories are no longer supported, as you should use a data collection for that.

Includes facade changes

The following methods in the Includes facade now return HtmlString objects instead of string:

• Includes::html()
• Includes::blade()
• Includes::markdown()

Notes:

• This means that you no longer need to use {!! !!} to render the output of these methods in Blade templates, instead just use {{ }}.
• If you have used the return value of these methods in custom code, you may need to adjust your code to work with the new return type.

For more information, see the RFC that proposed this change: #1734
The RFC was implemented in #1738

Remember to escape output if necessary

Note: Remember that this means that includes are no longer escaped by default, so make sure to escape the output if necessary, for example if the content is user-generated.

• (Use {{ e(Includes::html('foo')) }} instead of {{ Includes::html('foo') }} to escape the output, matching the previous behavior.)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.95%. Comparing base (8de4d37) to head (befd302).

Additional details and impacted files

@@            Coverage Diff             @@
##             2.x-dev    #1738   +/-   ##
==========================================
  Coverage      99.95%   99.95%           
- Complexity      1806     1810    +4     
==========================================
  Files            185      185           
  Lines           4786     4793    +7     
==========================================
+ Hits            4784     4791    +7     
  Misses             2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[emmadesilva]

Could we hook into the Blade @include directive? (And should we?)