fix #16561

[long76]

What kind of change does this PR introduce?
see #16561

Did you add tests for your changes?
no

Does this PR introduce a breaking change?

What needs to be documented once your changes are merged?
add globalObject option to page with amd library plugin

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: long76 (e45b75f)

[webpack-bot]

For maintainers only:

• This needs to be documented (issue in webpack/webpack.js.org will be filed when merged)
• This needs to be backported to webpack 4 (issue will be created when merged)

[long76]

please fix last 2 lint error i worked with ide github

[alexander-akait]

Please add tests

[long76]

Please add tests

ok i need some time

[long76]

Please add tests

done, param name changed to amdContainer

[alexander-akait]

@TheLarkInn Makse sense to look at this too, don't see somethinb bad here

[TheLarkInn]

LGTM. Thanks for the contribution 🚀

[alexander-akait]

@TheLarkInn Unmerged due 3 failing checks, but there are no problems with other tests

[long76]

@TheLarkInn Unmerged due 3 failing checks, but there are no problems with other tests

I can try add more tests for increasing coverage

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: long76 (e45b75f, 7b07887, 0239355, d371a69, 3db5440, eb7eb0b, e402736, 8c20d74)