Differential D10418

Highlight numbers in Python import names properly
ClosedPublic
Actions

Authored by sophiebits on Sep 5 2014, 12:15 AM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rPHUe29b637875fe: Highlight numbers in Python import names properly

Summary

This was a typo I made when porting the Pygments lexer.

Test Plan

Made a paste with "import webapp2_extras" and saw that the whole "webapp2_extras" was blue (unlike before, when only "webapp" was).

Diff Detail

Repository

rPHU libphutil

Branch

import-num

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 2436
Build 2440: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

sophiebits updated this revision to Diff 25070.Sep 5 2014, 12:15 AM

sophiebits retitled this revision from to Highlight numbers in Python import names properly.

sophiebits updated this object.

sophiebits edited the test plan for this revision. (Show Details)

sophiebits added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptSep 5 2014, 12:15 AM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

epriestley accepted this revision.Sep 5 2014, 12:15 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Sep 5 2014, 12:15 AM

Closed by commit rPHUe29b637875fe (authored by @spicyj, committed by @epriestley).

Harbormaster completed remote builds in B2436: Diff 25070.Sep 5 2014, 12:16 AM

Revision Contents
Changeset List

Path

Size

src/

lexer/

PhutilPythonFragmentLexer.php

4 lines

Diff 25070

View Options

src/lexer/PhutilPythonFragmentLexer.php

Highlight numbers in Python import names properlyClosedPublicActions