Add babel-plugin-transform-async-to-module-method to packageData.js

[hiroppy]

No description provided.

[hzoo]

oh this an interesting one - technically it's that new package but we have to support adding options.. yay 😅

{
  "plugins": [
-    ["@babel/transform-async-to-module-method"]
+    ["@babel/transform-async-to-generator", {
+      "module": "bluebird",
+      "method": "coroutine"
+    }]
  ]
}

Also very few people use this package but I guess figuring it out could be useful

[hiroppy]

Oops, sorry I forgot the option 😭

[hiroppy]

@hzoo hmm... It seems better to expand it as a character string? But maybe it is difficult.🤔

expect(upgradeDeps(depsFixture, VERSION)).toMatchSnapshot();

[noahlemen]

just came across this while browsing older PRs!

if i'm understanding this correctly, it looks like this is upgradeDeps trying to print { module: 'bluebird', method: 'coroutine' } as a dependency?

think this emerges from the handling of plugins that need to be replaced by two dependencies:

babel-upgrade/src/packageData.js

Line 89 in 00c5c2f

'babel-plugin-syntax-export-extensions': ['@babel/plugin-syntax-export-default-from', '@babel/plugin-syntax-export-namespace-from'],

babel-upgrade/src/upgradeDeps.js

Lines 24 to 25 in 00c5c2f

	if (Array.isArray(newPackageName)) {
	newPackageName.forEach(name => dependencies[name] = version);

we could check to make sure we only print strings to deps?

some stuff i was playing with in #55 might also relate here -- i was thinking about using some sort of object shape (as seen at https://github.com/babel/babel-upgrade/pull/55/files#diff-cf061c524d265adf7cfa52b35375795cR128) to describe deps that require more complicated transformations