Skip to content

feat: Add a simple code formatter#35

Open
AaronDewes wants to merge 5 commits intoClickism:devfrom
AaronDewes:auto-format
Open

feat: Add a simple code formatter#35
AaronDewes wants to merge 5 commits intoClickism:devfrom
AaronDewes:auto-format

Conversation

@AaronDewes
Copy link
Copy Markdown
Contributor

I'm not sure if this is something we want, but I think it's useful to have. It's not perfect, but formats most code to match the conventions we have and makes it more readable.

I'm not sure if this is something we want, but I think it's useful to have. It's not perfect, but formats most code to match the conventions we have and makes it more readable.
@Clickism
Copy link
Copy Markdown
Owner

I'm not sure if we should merge this to master, but nonetheless some feedback:

  • I would reccomend keeping blank lines as is because currently it removes all blank lines
  • Also, probably best to ignore comments because right now there are some bugs with arg0 stuff

@AaronDewes
Copy link
Copy Markdown
Contributor Author

The newline thing was unintentional. What is the arg0 problem with comments?

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 24, 2026

Deploy Preview for amazeingproject ready!

Name Link
🔨 Latest commit 19b8fed
🔍 Latest deploy log https://app.netlify.com/projects/amazeingproject/deploys/69c2d4ca4d51070008f4e138
😎 Deploy Preview https://deploy-preview-35--amazeingproject.netlify.app
📱 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 project configuration.

@Clickism
Copy link
Copy Markdown
Owner

Clickism commented Mar 24, 2026

I think actually its because of the colon after arg0, you could ignore colons in comments

@AaronDewes
Copy link
Copy Markdown
Contributor Author

Oh, right 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants