-
Notifications
You must be signed in to change notification settings - Fork 7
Fix apps in monorepo #18
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
Fix apps in monorepo #18
Conversation
jsamr
left a comment
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.
Thanks for your contribution!
bun.lockb
Outdated
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.
Bun looks like an interesting choice. But at the moment we have both yarn.lock and bun.lockdb
Is there any public place where you have discussed this preference?
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.
I used it out of habit and forgot to remove the generated files. I'll get on it.
0efc6ac to
ab1eb3b
Compare
5ZYSZ3K
left a comment
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.
Mostly minor issues
apps/website/docusaurus.config.js
Outdated
| trackingID: 'G-CYR1XDV25N', | ||
| anonymizeIP: true | ||
| }, | ||
| // Algolia search disabled - requires appId in Docusaurus v3 |
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.
Add something like TODO: remove or fix here to help find this place in the future
| const props = reflection.children?.filter( | ||
| (c) => c.kind === ReflectionKind.Property | ||
| (c) => | ||
| (c as any).kindString === 'Property' || c.kind === ReflectionKind.Property |
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.
not a fan of such a cast, add TODO: comment here
| const methods = reflection.children?.filter( | ||
| (c) => c.kind === ReflectionKind.Method | ||
| (c) => | ||
| (c as any).kindString === 'Method' || c.kind === ReflectionKind.Method |
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.
as above
| <TokenPlain>{reflection.name}</TokenPlain> | ||
| {renderTypeParameters( | ||
| reflection.typeParameter, | ||
| (reflection as any).typeParameter || |
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.
here and throughout this file as well
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## chore/migrate-render-html-with-history #18 +/- ##
==========================================================================
- Coverage 99.30% 98.65% -0.66%
==========================================================================
Files 57 140 +83
Lines 1153 2076 +923
Branches 315 636 +321
==========================================================================
+ Hits 1145 2048 +903
- Misses 8 27 +19
- Partials 0 1 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
9e12a16
into
native-html:chore/migrate-render-html-with-history
No description provided.