loadModules examples in the README - async/await?

To me, the way @jwasilgeo is using loadModules with async/await here is much more readable. Do others agree? Now that async await is pretty mainstream (sorry IE11), I’m wondering if we should switch all our examples in the README to use this style (we could still include one Promises style example just to cover all bases)

[tomwayson]

I think we should do this. It’s time.

[gavinr]

One note that I’ve realized since I posted the original issue:

sometimes part of the “simplicity” of async/await examples is actually just because the promise-based includes error handling where the async/await example [incorrectly] leaves it out.

If we take an example from our actual README:

// component.js
import { loadModules } from 'esri-loader';

// this will lazy load the ArcGIS API
// and then use Dojo's loader to require the map class
loadModules(['esri/map'])
.then(([Map]) => {
  // create map with the given options at a DOM node w/ id 'mapNode'
  let map = new Map('mapNode', {
    center: [-118, 34.5],
    zoom: 8,
    basemap: 'dark-gray'
  });
})
.catch(err => {
  // handle any script or module loading errors
  console.error(err);
});

It would be tempting and satisfying to change it to async/await without error handling:

// component.js
import { loadModules } from 'esri-loader';

// this will lazy load the ArcGIS API
// and then use Dojo's loader to require the map class
const [Map] = await loadModules(['esri/Map']);

// create map with the given options at a DOM node w/ id 'mapNode'
let map = new Map('mapNode', {
  center: [-118, 34.5],
  zoom: 8,
  basemap: 'dark-gray'
});

… but that’s not equivalent to the original promise-based example! Making it equivalent (including error catching):

// component.js
import { loadModules } from 'esri-loader';

// this will lazy load the ArcGIS API
// and then use Dojo's loader to require the map class
try {
  const [Map] = await loadModules(['esri/Map']);

  // create map with the given options at a DOM node w/ id 'mapNode'
  let map = new Map('mapNode', {
    center: [-118, 34.5],
    zoom: 8,
    basemap: 'dark-gray'
  });

} catch (err) {
  // handle any script or module loading errors
  console.error(err);
}

… which makes it closer in complexity to the promise-based example. I would argue the async/await-example-with-try-catch example is still slightly better readability, but only marginally.