top level await example updated with await keyword

[xkeshav]

in top level example file; update syntax with top level await

[github-actions]

It looks like this is your first pull request. 🎉 Thank you for your contribution! One of the project maintainers will triage and assign the pull request for review. We appreciate your patience. To safeguard the health of the project, please take a moment to read our code of conduct.

[xkeshav]

any update here ?

[bsmth]

Hi @xkeshav sorry I missed this one, I'm going through repositories to see if there's anything we overlooked. @Josh-Cena are you interested in having a look?

[Josh-Cena]

Hello, unfortunately this change is anti-performance and unnecessary. We are using import on demand on purpose, and even if we don't, we will be using static imports instead of await import(...) at the top level, which you should virtually never do. If you check out the accompanying content text:

Then we'll create a module called getColors.js which uses a fetch request to load the colors.json file and return the data as an object.

The top level await is actually in the getColors.js file.

[bsmth]

Thanks a lot for chiming in, Josh. @xkeshav I hope that clarifies things for you. I'm going to close this PR now but if you have some more questions of if you think there's a mistake, please let us know here. Thank you :)

[xkeshav]

Got it. Thank you for the detailed explanation.