docs: Explain how ngsw.json is generated.

[Arjen-1]

ngsw.json is the manifest file which is generated at build time ( based on ngsw-config.json ). This explains this a bit clearer.
At first, I thought that it was a wrongly named filename. Not is it explicitly described.

PR Type

Docs

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation content changes
• angular.io application / infrastructure changes
• Other... Please describe:

What is the current behavior?

Its not explained how ngsw is created and how it differs from ngsw-config

What is the new behavior?

It explains it.

Does this PR introduce a breaking change?

• Yes
• No

Other information

Still a proces of my first contribution!

[Arjen-1]

I'd love to know if my first contribution has a green light ! :D

[gkalpak]

Thx, @Arjen-1 ❤️
I made a small suggestion - otherwise lgtm 👍

[Arjen-1]

Thanks @gkalpak , the suggestion looks good!

[mary-poppins]

You can preview ae49391 at https://pr44823-ae49391.ngbuilds.io/.
You can preview 21c1180 at https://pr44823-21c1180.ngbuilds.io/.

[gkalpak]

Thx for making the update, @Arjen-1 👍
LGTM

[gkalpak]

Oh, wait! Can you please squash the commits into one?

[Arjen-1]

@gkalpak , I believe it's squashed into one commit now. Thanks for the tip!

[gkalpak]

The for squashing, @Arjen-1. I'm afraid there are some more changes needed, though 😅
The commit message has to follow our commit message guidelines. So, could you change it again to something like your original commit message? 🙏
For example:

docs: explain how `ngsw.json` is generated

`ngsw.json` is the manifest file which is generated at build time (based
on `ngsw-config.json`). This commit explains this a bit clearer. At
first, I thought that it was a wrongly named filename. Now is it
explicitly described.

[gkalpak]

I forgot to mention that we also need you to sign the CLA before we can merge this PR, @Arjen-1. See here for more details.

[Arjen-1]

@gkalpak , I've done the CLA step and restored the correct commit message. Any more steps that need to be done?
If so, no problem!
Of not, yay! I almost made my first contribution!

[AndrewKushnir]

@Arjen-1 thanks for addressing the feedback. I've noticed that the setup CI job is failing and I've rebased the PR on top of the most recent version of the master branch. I'm adding this PR to the merge queue. Thanks for improving Angular docs!

[mary-poppins]

You can preview d0c5449 at https://pr44823-d0c5449.ngbuilds.io/.

[jessicajaniuk]

This PR was merged into the repository by commit 9f5edce.

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}