Skip to content

Create editorconfig file #35

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

cheny0
Copy link
Contributor

@cheny0 cheny0 commented Jul 24, 2025

I combined the file from the status-website repo (https://github.com/status-im/status-website/blob/main/.editorconfig) and some recommendations from ChatGPT

@cheny0 cheny0 requested a review from jorge-campo July 24, 2025 09:59
@cheny0 cheny0 self-assigned this Jul 24, 2025
@cheny0 cheny0 added the documentation Improvements or additions to documentation label Jul 24, 2025
@cheny0 cheny0 linked an issue Jul 24, 2025 that may be closed by this pull request
@cheny0 cheny0 added deliverables Any other project deliverable and removed documentation Improvements or additions to documentation labels Jul 25, 2025
Copy link
Contributor

@jorge-campo jorge-campo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cheny0 , any reason to use an indent size of 2 ?

For code files, where code is indented multiple times, 2 is ideal to avoid the code to expand/wrap on the page too frequently. However, for text (Markdown), I would recommend using 4 to spot indentation easier.

I believe 4 it's also the by-default VSC default indent size.

@cheny0
Copy link
Contributor Author

cheny0 commented Jul 29, 2025

@jorge-campo, because indent size of 2 is used for status website repo and we've been using it for Status docs. ChatGPT also mentioned that many Markdown editors and static site generators (like Docusaurus, GitBook, and others) default to 2-space indentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deliverables Any other project deliverable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create the EditorConfig file
2 participants