Kent C. Dodds

Results 893 comments of Kent C. Dodds

Just a drive-by comment. If this approach still works then why would you need another solution? ๐Ÿ˜…

I've seen this in the editor, but not when running the CLI. I'm not sure what is incorrectly configured, but I'd love to find out how to properly handle this.

I'm not sure whether we need to do this or not because I don't have this issue regularly at all. So I think we'll just close this for now.

Can I safely assume that this isn't going to work based on our conversation on [๐•](https://x.com/kettanaito/status/1987883638097154328)?

I intentionally made the tests run against the dev database because it helps find cleanup issues (when you go into develop after running the tests you find you've got a...

I also see this in my logs and it's really confusing.

I was about ready to merge this and then I noticed when I started the dev server we get a bunch of warnings from MSW I would like you to...

@cursor, the CI build has an error. Please fix this: โœ— Build failed in 1.40s [commonjs--resolver] Server-only module referenced by client '#app/middleware.server.ts' imported by route 'app/routes/settings+/profile.tsx' React Router automatically removes...

@cursor, several of the playwright tests are failing in CI with the following output. Please fix: 8 failed [chromium] โ€บ tests/e2e/2fa.test.ts:5:1 โ€บ Users can add 2FA to their account and...

@cursor, could you review the comments and try to get things working. Don't worry about the modified seo suggestion. We can look into that in another PR.