Skip to content

Conversation

@Aimocs
Copy link

@Aimocs Aimocs commented Jun 7, 2024

No description provided.

@Dylan700
Copy link
Owner

Hi @Aimocs, thanks for contributing! Could you please run yarn lint and yarn test on your code? I believe you need to fix a few indentation errors to succeed the checks.

Seems the integration tests are failing but I don't think that's you. I believe some package I was using got updated and changed the behavior of the workflow somehow 🫠 .

If you can get the lint and unit tests passing I'll test e2e manually then we can merge. Let me know if you have any questions or need help!

@Aimocs
Copy link
Author

Aimocs commented Jun 10, 2024

i ran yarn lint and yarn test
yarn lint all indentation errors fixed but
yarn test says 5 failed out of 6 and 7 failed out of 20

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