Updating localhost removal logic (#280)

dojo / webpack-contrib

Plugins and loaders for webpack used with Dojo

Other

4 stars 19 forks source link

Closed rorticus closed 4 years ago

rorticus commented 4 years ago

Type: feature

The following has been addressed in the PR:

[x] There is a related issue
[x] All code has been formatted with prettier as per the readme code style guidelines
[x] Unit or Functional tests are included in the PR

Description:

Updating localhost removal to only remove "http://localhost:1234" from attributes that can contain urls, rather than any place in the content.

Resolves #280

rorticus commented 4 years ago

After discussions with @matt-gadd and @agubler , it looks like the localhost removal code isn't even needed anymore.