Closed Bug 1562572 Opened 6 years ago Closed 6 years ago

CamelCase file and folder names in devtools/shared/protocol

Categories

(DevTools :: General, task, P3)

task

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: jdescottes, Assigned: jdescottes)

References

Details

The convention in devtools is to use lowercase + hyphenated file names for all files except react components.

We don't really have a set rule for this, and we have a few old folders that don't follow the rule, but 95% of the /devtools code follows this rule today. I would like to keep the codebase as consistent as possible.

And I'm not against going for CamelCase for all modules, but in this case we should do it everywhere.

Priority: -- → P3

Well, looks like we will have to live with it. Anyway I tried to do it, I am facing case folding issues.
I could submit working commits, but they will be a pain at every rebase/histedit/...

Unless we use really different class names for all the files here, I don't think there's anything we can do (or at least not in a decent amount of time).

Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.