Skip to content

Chore: remove exclusive test and add prop type for test component#277

Merged
pixelbandito merged 2 commits intomasterfrom
chore-remove-exclusive-test-and-add-prop-type-for-test-component
Oct 27, 2020
Merged

Chore: remove exclusive test and add prop type for test component#277
pixelbandito merged 2 commits intomasterfrom
chore-remove-exclusive-test-and-add-prop-type-for-test-component

Conversation

@pixelbandito
Copy link
Copy Markdown
Contributor

@pixelbandito pixelbandito commented Oct 26, 2020

We were running prettier during pre-commit hooks, but we weren't checking eslint.

I noticed because there was an exclusive test checked in, and we have a rule about that.

So, this change:

  • Puts eslint back in the lint-staged task for js/jsx/ts/tsx files.
  • Removes .only from a test
  • Adds a TS type def for the TestComponent in that test.

@pixelbandito pixelbandito force-pushed the chore-remove-exclusive-test-and-add-prop-type-for-test-component branch from 27e17c4 to 000b203 Compare October 26, 2020 18:54
@pixelbandito pixelbandito requested review from dkordik, nvmusoke and wendigolabs and removed request for dkordik October 26, 2020 18:55
@pixelbandito pixelbandito force-pushed the chore-remove-exclusive-test-and-add-prop-type-for-test-component branch from 000b203 to 4159e97 Compare October 27, 2020 16:12
@pixelbandito pixelbandito merged commit cd86348 into master Oct 27, 2020
@pixelbandito pixelbandito deleted the chore-remove-exclusive-test-and-add-prop-type-for-test-component branch October 27, 2020 16:16
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