-
Notifications
You must be signed in to change notification settings - Fork 712
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
typescript(vx-pattern): re-write package in TypeScript #503
Conversation
/** Fill applied to circles. */ | ||
fill?: string; | ||
/** className applied to circles. */ | ||
className?: string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How are the props ordered by?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is not enforced ordering, I enabled several new rules in the nimbus
upgrade but decided that was too big an undertaking / can't auto-fix.
cc2919a
to
2d01d26
Compare
…in switch/don't break
2d01d26
to
bbccc9f
Compare
|
||
describe('<PatternWaves />', () => { | ||
beforeEach(() => { | ||
global.console.error = jest.fn(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
highlighting that nimbus
mocks out console.error
to throw, so we don't have to do this ourselves anymore.
🚀 Enhancements
This PR builds off of #488 which added TypeScript build support, and converts the
@vx/pattern
package toTypeScript
.Closes #502.
Testing
/patterns
💥 Breaking Changes
PatternOrientation
is no longer thedefault
export of@vx/patterns/lib/constants
and is instead a named export.PatternOrientation
is used as the export name if importing from the index:import { PatternOrientation } from '@vx/pattern'
@hshoff @schillerk @milesj @techniq @kristw