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

feat(rome_service): apply formatter to code actions #4324

Merged
merged 1 commit into from
Apr 4, 2023

Conversation

ematipico
Copy link
Contributor

@ematipico ematipico commented Mar 26, 2023

Summary

With this PR, the code emitted by the analyzer actions is now formatted using Rome's internal formatter. After all the analyzer actions have been committed to the current code, the formatting is applied at the end.

If the formatter is disabled, no formatting is applied.

Test Plan

Existing tests should be updated.

Documentation

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

@netlify
Copy link

netlify bot commented Mar 26, 2023

Deploy Preview for docs-rometools ready!

Name Link
🔨 Latest commit 65b1723
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/642bdf210696cb0008054cc1
😎 Deploy Preview https://deploy-preview-4324--docs-rometools.netlify.app/playground
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions github-actions bot added A-CLI Area: CLI A-Formatter Area: formatter A-Project Area: project configuration and loading labels Mar 26, 2023
@ematipico ematipico force-pushed the feat/formatted-code-actions branch 2 times, most recently from 0884dbc to 5799e4f Compare April 2, 2023 14:10
@ematipico ematipico marked this pull request as ready for review April 2, 2023 14:10
@ematipico ematipico force-pushed the feat/formatted-code-actions branch from 5799e4f to bb54df8 Compare April 3, 2023 12:49
@github-actions github-actions bot added the A-LSP Area: language server protocol label Apr 3, 2023
@ematipico ematipico force-pushed the feat/formatted-code-actions branch from bb54df8 to b0a7927 Compare April 4, 2023 08:25
@ematipico ematipico force-pushed the feat/formatted-code-actions branch from b0a7927 to 65b1723 Compare April 4, 2023 08:26
@ematipico ematipico merged commit e82c977 into main Apr 4, 2023
@ematipico ematipico deleted the feat/formatted-code-actions branch April 4, 2023 10:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-CLI Area: CLI A-Formatter Area: formatter A-LSP Area: language server protocol A-Project Area: project configuration and loading
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant