docs(webpack): fix for tsconfig.json provided in example

[skryvets]

Pull request for this issue.

[googlebot]

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

---

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
• If you signed the CLA as a corporation, please let us know the company's name.

[googlebot]

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

---

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
• If you signed the CLA as a corporation, please let us know the company's name.

[googlebot]

CLAs look good, thanks!

[skryvets]

I signed it!

[Foxandxss]

We don't need exclude without files and why do we need typeRoots? I am totally in for the lib, I wanted to add that with an update but forgot.

[skryvets]

Hello @Foxandxss ,

1. I've added exclude in order to not process all files inside node_modules.
2. I thought that typeRoots actually can help solve the issue. But I've just verified and add "lib": ["es2015", "dom"], is enough.

Sorry. for this.
As far as I understand we just need to add "lib": ["es2015", "dom"],. Can I modify it and submit updated version in this pull request?

Best regards,
Siarhei

[Foxandxss]

Sure you can.

The node_modules exclusion is only needed if you specify the files by hand, since we do not, node_modules is excluded automagically.

[skryvets]

@Foxandxss,
Thank you for the explanation. I didn't know about that.
I've updated my changes.

Best regards,
Siarhei

[wardbell]

Would you please remove this removeComments setting too. That is the default setting and we don't want it here. I thought I'd removed it everywhere but I guess I missed this one.

Then I can merge your PR.

[skryvets]

Hello @wardbell ,

I've just removed removeComments setting and pushed my changes.
Sorry for the delay with this.

Best regards,
Siarhei

[Foxandxss]

Thank you.