When default exporting a Flow Enum, do not append semicolon

[gkz]

This is a bug fix (missed this during initial support). Like function declarations and class declarations, Flow Enum declarations should not be trailed by a semicolon.

• I’ve added tests to confirm my change works.
• (If the change is user-facing) I’ve added my changes to changelog_unreleased/*/pr-XXXX.md file following changelog_unreleased/TEMPLATE.md.
• I’ve read the contributing guidelines.

✨Try the playground for this PR✨

[fisker]

@gkz Do you know will this format into invalid code?

Prettier pr-8768
Playground link

--parser flow
--no-semi

Input:

export default enum B {}

[].forEach(() => 0)

Output:

export default enum B {}

;[].forEach(() => 0)

[gkz]

That behavior is fine, the semicolon does not create invalid code, it was just not necessary.

[gkz]

Looks like we do the same thing for classes:

export default class B {}

[].forEach(() => 0)

=>

export default class B {}

;[].forEach(() => 0)

https://deploy-preview-8768--prettier.netlify.app/playground/#N4Igxg9gdgLgprEAuEcAeAHCAnGACAEzgDMBDAVwBt8xLSBnevAIT2AF8AdKbgbQF0AdMRwBRUmAAWACmkBKPAF4AfHgAMckABoQEDDACW0eslCls2CAHcACuYQmUpSldIBPEzoBG2CQGs4GABlDAkDKABzZBhscjgdOABbLzgCIgIAGVJI8lIIuAAxHETSGENI5BAKGAhtEEkYRMoAdUkDeHpQsDggh3aDADd2t0qwRjrw+jhcG18IkuQySimdACt6NGZfMADg0kS4DPC4RecVkHW0IPCIyjgARXIIeFPl+JBQ7CnsSuJKazqGGw4RgzQMBBgkmQAA41DogRAps1fBhKkC4N8BicdABHJ7wWZ6RxVegAWigcFSqTq2DgeIMtNmeQWSCW5ymiQMr3ZNzuj2eJ1ZZ3eMFIXjBEKhSAATDoYqQDJQbgBhCCJFmoegAVjq5CmABUxY42e8BnEAJJQIiwIJgYH6ACCVqCMDcd25cHY7CAA

[fisker]

Okay, thanks.

[thorn0]

@fisker That's a documented behavior: https://prettier.io/docs/en/rationale.html#semicolons

[fisker]

I know that, I was afraid that ; break things.