Highlight numbers in Python import names properly
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Nov 21, 3:22 AM

	Unknown Object (File)
	Sun, Nov 17, 7:39 AM

	Unknown Object (File)
	Wed, Oct 30, 6:42 AM

	Unknown Object (File)
	Oct 22 2024, 8:29 PM

	Unknown Object (File)
	Oct 22 2024, 2:25 PM

	Unknown Object (File)
	Oct 22 2024, 9:10 AM

	Unknown Object (File)
	Oct 8 2024, 11:58 PM

	Unknown Object (File)
	Sep 23 2024, 1:02 PM

View All Files

Subscribers

epriestley

Korvin

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

Lint

Lint Skipped

Unit

Tests Skipped

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 25071

View Options

src/lexer/PhutilPythonFragmentLexer.php

Highlight numbers in Python import names properlyClosedPublicActions