1505309 - Remove unused netmonitor webpack config as well as devtools/shared/webpack

Status: RESOLVED DUPLICATE of bug 1567084

(DevTools :: Netmonitor, enhancement, P3)

Description

[Alexandre Poirot [:ochameau]]

The netmonitor is no longer supporting its launchpad/webpack setup and it seems to be the only used of the whole devtools/shared/webpack folder:
https://searchfox.org/mozilla-central/search?q=shared%2Fwebpack&case=true&regexp=false&path=

So I imagine we can get rid of all of that.

Comment 1

[Manish [:manishkk]]

Need to remove these files also?
https://searchfox.org/mozilla-central/source/devtools/client/shared/webpack

Comment 2

[Alexandre Poirot [:ochameau]]

Yes.
As well as:
devtools/client/netmonitor/webpack.config.js
And a reference here:
https://searchfox.org/mozilla-central/source/.eslintignore#104

And some other stuff also related to launchpad:
https://searchfox.org/mozilla-central/source/devtools/client/netmonitor/yarn.lock
https://searchfox.org/mozilla-central/source/devtools/client/netmonitor/webpack.config.js
https://searchfox.org/mozilla-central/source/devtools/client/netmonitor/bin
https://searchfox.org/mozilla-central/source/devtools/client/netmonitor/launchpad.js
https://searchfox.org/mozilla-central/source/devtools/client/netmonitor/package.json
https://searchfox.org/mozilla-central/source/devtools/client/netmonitor/.babelrc

And this section of the README:
https://searchfox.org/mozilla-central/source/devtools/client/netmonitor/README.md#17-27

For the patch, I would suggest asking the review to :Honza to double check.

Comment 3

[Alexandre Poirot [:ochameau]]

The whole "bin" and "configs" folders should be removed.

Comment 4

[Neha]

Hello Jan,
I would like to work on this.
Could it be assigned to me?
Thanks!

Comment 5

[[:Towkir] Ahmed]

Looks like this is no longer valid. Could you close this ?

Comment 6

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Correct, this has been already done.

Thanks for the note!
Honza