Closed Bug 1037671 Opened 11 years ago Closed 9 years ago

Don't hide require() in order to get syntax highlight working

Categories

(Marketplace Graveyard :: Code Quality, defect, P5)

Product:
Marketplace Graveyard
Component:
Code Quality
Version:
2014-Q2
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: delza, Unassigned)

Details

(Whiteboard: [marketplace-transition])

For the File View and Compare feature in Reviewer Tools, we actually set `require = undefined`, apparently in order to get the syntax highlighting working. There are lots of great syntax highlighters out there, and if the one we're using has this problem and we can't work around it less invasively than that then perhaps we could use a different one. We currently have a work-around in place to allow the use of `require()` where we need it in devreg/devhub.js, but it would be better to not resort to the hack of undefining require at all.
Priority: -- → P5
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Whiteboard: [marketplace-transition]
You need to log in before you can comment on or make changes to this bug.