Skip to content
This repository has been archived by the owner on Jun 12, 2024. It is now read-only.

run eslint + fix 2 tests #22

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

run eslint + fix 2 tests #22

wants to merge 1 commit into from

Conversation

ashishkeshan
Copy link
Contributor

Ran ESLint and fixed 2 tests to just check whether it contains a ✅ or ❌ to signify pass/fail.

@ashishkeshan ashishkeshan requested a review from a team as a code owner December 21, 2023 17:49
JessRudder
JessRudder approved these changes Dec 22, 2023
View reviewed changes
Copy link

@JessRudder JessRudder left a comment

Choose a reason for hiding this comment

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

Whoa! eslint has opinions...it makes it hard to know what actually changed and what was just formatting. 🤣 I think I got there though.

This looks solid!

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

@JessRudder JessRudder JessRudder approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ashishkeshan @JessRudder