CodeGen reformats package.json after prettier is run

[trivikr]

Describe the bug

• CodeGen reformats package.json after prettier is run
• Prettier is run before copying to clients folder

  aws-sdk-js-v3/scripts/generate-clients/index.js

  Lines 25 to 26 in 6c0af9c

  	await prettifyCode();
  	await copyToClients(clientsDir);

• In chore: copy package.json properly #900, the package.json was reformatted using JSON.stringify

  aws-sdk-js-v3/scripts/generate-clients/copy-to-clients.js

  Line 90 in 6c0af9c

  writeFileSync(destSubPath, JSON.stringify(mergedManifest, null, 2));

• This is not an issue right now, as codegen is manually run and lint-staged runs prettier on package.json on precommit. But this will become an issue when codegen is run in release scripts.

To Reproduce (observed behavior)
Run yarn generate-clients -m models, and notice that it updates all package.json files

Expected behavior
The codegen should emit prettified code

[trivikr]

The only difference between existing codegen and command line prettier is no newline at end of file

[trivikr]

Fixed in #913

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.