Skip to content

blog/twie#129 - #1417

Draft
DadeSko wants to merge 3 commits into
mainfrom
blog/twie#129
Draft

blog/twie#129#1417
DadeSko wants to merge 3 commits into
mainfrom
blog/twie#129

Conversation

@DadeSko

@DadeSko DadeSko commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

add src/content/blog/this-week-in-effect/129/index.mdx

@DadeSko DadeSko self-assigned this Jul 30, 2026
@DadeSko
DadeSko requested a review from mirelaprifti July 30, 2026 18:05
@IMax153

IMax153 commented Jul 30, 2026

Copy link
Copy Markdown
Member

@DadeSko - we've introduced some additional CI checks to make sure files are formatted and structured as we would expect.

You can see the failing checks above - please run pnpm fmt:fix in a shell in the root of the project and commit and push the changes to resolve the issues.

@DadeSko

DadeSko commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

@DadeSko - we've introduced some additional CI checks to make sure files are formatted and structured as we would expect.

You can see the failing checks above - please run pnpm fmt:fix in a shell in the root of the project and commit and push the changes to resolve the issues.

Done! ty @IMax153 🙏🏻 I still see some checks failing though, let me know if I caused any other issue

@IMax153

IMax153 commented Jul 30, 2026

Copy link
Copy Markdown
Member

@DadeSko - you can ignore the failing CI job that remains - its a new one I introduced and may have issues. I'll investigate.

@DadeSko

DadeSko commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

Got it! thx again @IMax153 !

@IMax153

IMax153 commented Jul 30, 2026

Copy link
Copy Markdown
Member

Ah @DadeSko I realized it's because this PR is behind the main branch.

We can ignore that failing CI step and just merge this PR tomorrow - if you need help just ping me.

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