Skip to content

CONTRIBUTING.md

Latest commit

 

History

History
37 lines (27 loc) · 1.45 KB

CONTRIBUTING.md

File metadata and controls

37 lines (27 loc) · 1.45 KB

CONTRIBUTING

When contributing to this project, please keep in mind the following goals:

  • The hook must remain as simple as possible. It's only a "proxy" to a ResizeObserver instance, and shouldn't add features that the resize observer doesn't do.
  • All features must be covered with test(s).

It's also best to first submit an issue, before creating a pull request so that the required feature can be discussed, as well as the actual implementation.

Adding a New Feature

  • Open an issue, so that a discussion can take place,
  • Once discussed, add the changes to src/index.ts accordingly (make sure TS types are respected as well),
  • Add new test(s) to cover the new feature in: test/testing-lib.tsx. Ignore the other test files, as they're fairly old and much harder to add new ones to them compared to just using react testing lib.
  • Run all the tests to ensure there are no regressions: yarn test,

Using Watch Modes While Developing

There are other test-related scripts in package.json, to enhance the developer experience. While making changes you might want to watch the source files, and build them automatically, as well as having Karma run a (non-headless) Chrome instance every time a change was made.

To do that:

  • Run yarn src:watch in a terminal tab
  • Run KARMA_BROWSERS=Chrome yarn karma:watch in another.

Don't forget to run yarn test at the end once you're done with everything, to make sure the new code is tested for regressions.