Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

fix(rome_js_formatter): Union in mapped type #4100

Merged
merged 1 commit into from
Dec 23, 2022
Merged

Conversation

MichaReiser
Copy link
Contributor

Summary

Fixes #4074

Test Plan

I verified the updated snapshot to ensure it matches Prettier's output.

Documentation

  • The PR requires documentation
  • I will create a new PR to update the documentation

@MichaReiser MichaReiser added A-Formatter Area: formatter L-JavaScript Langauge: JavaScript labels Dec 23, 2022
@MichaReiser MichaReiser added this to the Next milestone Dec 23, 2022
@netlify
Copy link

netlify bot commented Dec 23, 2022

Deploy Preview for docs-rometools canceled.

Name Link
🔨 Latest commit 8d5b07e
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/63a58ba97461a00008152a5e

@MichaReiser MichaReiser force-pushed the fix/union-mapped-type branch 2 times, most recently from ee9421f to b7f6f44 Compare December 23, 2022 10:56
@MichaReiser MichaReiser merged commit 84812f0 into main Dec 23, 2022
@MichaReiser MichaReiser deleted the fix/union-mapped-type branch December 23, 2022 11:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Formatter Area: formatter breaking L-JavaScript Langauge: JavaScript
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

🐛 Indentation differs between rome and prettier for multiline mapped types with union key
2 participants