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

docs: change indent-style from tabs to tab #3657

Merged
merged 1 commit into from
Nov 11, 2022
Merged

docs: change indent-style from tabs to tab #3657

merged 1 commit into from
Nov 11, 2022

Conversation

lolopinto
Copy link
Contributor

Summary

using tabs gives the following error so updating the docs:

Error: failed to parse argument '--indent-style': failed to parse 'tabs': Value not supported for IndentStyle

Test Plan

ran

rome format --indent-style tab .

and it worked

@lolopinto
change indent-style from tabs to tab
45610a3 
using tabs gives the following error:

```
Error: failed to parse argument '--indent-style': failed to parse 'tabs': Value not supported for IndentStyle
```
@lolopinto lolopinto requested a review from a team November 10, 2022 20:07
@netlify
Copy link

netlify bot commented Nov 10, 2022 •
edited
Loading

✅ Deploy Preview for docs-rometools ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 45610a3
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/636d59e9de831300098ab377
😎 Deploy Preview https://deploy-preview-3657--docs-rometools.netlify.app/formatter
📱 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.

@MichaReiser
Copy link
Contributor

Whoops. Thank you!

@MichaReiser MichaReiser added the A-Website Area: website and documentation label Nov 11, 2022
@MichaReiser MichaReiser merged commit ddb99ac into rome:main Nov 11, 2022
jeysal added a commit to jeysal/rometools that referenced this pull request Nov 11, 2022
@jeysal
Merge remote-tracking branch 'upstream/main' into no-dupe-keys
725d684 
* upstream/main:
  fix(website): fix grammar mistake (rome#3666)
  doc(website): Use `pnpm rome` and `yarn rome` instead of longer versions
  docs: change indent-style from tabs to tab (rome#3657)
  Revert "[website] Port docs to astro" (rome#3662)
  [website] Port docs to astro (rome#3659)
@jamesdury jamesdury mentioned this pull request Apr 18, 2023
Merged
3 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@MichaReiser MichaReiser MichaReiser approved these changes

Assignees
No one assigned
Labels
A-Website Area: website and documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lolopinto @MichaReiser