Skip to content

🚨 lint and format www/#1088

Merged
cowboyd merged 2 commits intov4from
lint-and-format-www
Jan 22, 2026
Merged

🚨 lint and format www/#1088
cowboyd merged 2 commits intov4from
lint-and-format-www

Conversation

@cowboyd
Copy link
Copy Markdown
Member

@cowboyd cowboyd commented Jan 22, 2026

Motivation

The www/ webapp was a bit of wild west because linting and formatting were turned off resulting in the style in the web app being all over the place.

Approach

This turns it back on and fixes and formats the code. There were two files that were causing deno lint to crash, so they've been excluded.

  • routes/guides-route.tsx
  • components/api/api-page.tsx

We can follow up with the deno team to find out why these are crashing later.

@cowboyd cowboyd requested review from jbolda and taras January 22, 2026 01:04
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Jan 22, 2026

Open in StackBlitz

npm i https://pkg.pr.new/thefrontside/effection@1088

commit: 56e61d3

The www/ webapp was a bit of wild west because linting and formatting
were turned off.

This turns it back on and fixes and formats the code. There were two
files that were causing `deno lint` to crash, so they've been excluded.

- routes/guides-route.tsx
- components/api/api-page.tsx

We can follow up with the deno team to find out why these are crashing
later.
Copy link
Copy Markdown
Member

@taras taras left a comment

Choose a reason for hiding this comment

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

👍

@cowboyd cowboyd merged commit d9788cc into v4 Jan 22, 2026
15 checks passed
@cowboyd cowboyd deleted the lint-and-format-www branch January 22, 2026 05:38
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