Closed Bug 1625495 Opened 4 months ago Closed 4 months ago

Remove devtools/shared/client folder

Categories

(DevTools :: General, task, P3)

task

Tracking

(firefox76 fixed)

RESOLVED FIXED
Firefox 76
Tracking Status
firefox76 --- fixed

People

(Reporter: jdescottes, Assigned: jdescottes)

References

Details

Attachments

(1 file)

The devtools/shared/client folder is confusing in our current folder layout.
We have:

  • devtools/shared: modules shared between client & server, protocol, specs
  • devtools/client/shared: shared ui components and widgets
  • devtools/shared/client: contains a few files that are in theory client files, but which are needed in some tests

I propose to move all the files of devtools/shared/client in preparation for the following RFC: https://bugzilla.mozilla.org/show_bug.cgi?id=1623937

If we go through the RFC above, such files would be in devtools/client, while frontend files would move to devtools/frontend (or devtools/client/frontend).

But in the meantime, devtools/shared/client is just confusing, especially considering it contains one of the most important files for the client: devtools-client.js

Depends on D68599

This changeset moves devtools-client.js and constants.js to devtools/client and removes devtools/shared/client

Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/414695ea5cd7
Remove devtools/shared/client folder r=daisuke,ochameau,nchevobbe
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 76
You need to log in before you can comment on or make changes to this bug.