Page MenuHomePhorge

Remarkup code blocks: guess language from "name="
ClosedPublic

Authored by valerio.bozzolan on Mar 25 2024, 07:41.
Tags
None
Referenced Files
F2184647: D25560.diff
Thu, May 9, 07:08
Unknown Object (File)
Mon, May 6, 14:00
Unknown Object (File)
Fri, May 3, 21:34
Unknown Object (File)
Fri, May 3, 21:34
Unknown Object (File)
Fri, May 3, 21:34
Unknown Object (File)
Thu, May 2, 10:00
Unknown Object (File)
Thu, Apr 25, 10:48
Unknown Object (File)
Thu, Apr 25, 10:48

Details

Summary

The file name is a sufficient source of information, if available.

Closes T15729

Test Plan

The new unit test is green. Old unit tests are green.

Diff Detail

Repository
rP Phorge
Branch
remarkup-lang-guess-from-name
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1122
Build 1122: arc lint + arc unit

Event Timeline

avivey added inline comments.
src/infrastructure/markup/blockrule/PhutilRemarkupCodeBlockRule.php
157–159

Better move this into the if (empty($options['lang']))) block below.

src/infrastructure/markup/blockrule/PhutilRemarkupCodeBlockRule.php
157–159

(Premising that it was a feature - kind of waterfall attempt - since guessFilenameExtension() can return NULL)

This revision is now accepted and ready to land.Mar 26 2024, 10:19