Closed ritch closed 4 months ago
The recent updates introduce several modifications across the app/packages/operators
module, primarily focusing on refining the use of hooks, enhancing type definitions, and updating configurations. Key changes include disabling a specific ESLint rule, adding dependencies to useEffect
hooks, refining method signatures, and updating type declarations. Additionally, minor adjustments were made to GitHub workflows by removing unnecessary installation steps.
File Path | Change Summary |
---|---|
app/packages/operators/.eslintrc |
Introduced configuration to disable the react-hooks/rules-of-hooks rule. |
app/packages/operators/babel.config.js |
Added a global comment for the module. |
app/packages/operators/src/OperatorInvocationRequestExecutor.tsx |
Added dependencies to the useEffect hook in the RequestExecutor function. |
app/packages/operators/src/built-in-operators.ts |
Updated method signatures and return types for several classes. |
app/packages/operators/src/types.ts |
Updated Property constructor and added new properties to PropertyOptions . |
app/packages/operators/src/OperatorPalette.tsx |
Changed SubmitButtonOption type from locally scoped to exported. |
app/packages/operators/src/components/OperatorPromptFooter.tsx |
Updated type of submitButtonOptions in OperatorFooterProps to use SubmitButtonOption[] . |
app/packages/operators/src/operators.ts |
Added new types and updated method signatures in multiple classes. |
.github/workflows/e2e.yml |
Removed pip install fiftyone-db-ubuntu2204 from the Install fiftyone job. |
.github/workflows/test.yml |
Removed pip install fiftyone-db-ubuntu2204 from the Install fiftyone job. |
Amidst the code, where changes bloom, A rabbit hops from room to room. With hooks refined and types in place, Our project moves at a steady pace. In workflows, clutter swept away, We code anew, come what may. 🐇✨
[!WARNING]
Review ran into problems
Problems (1)
* Git: Failed to clone repository. Please contact CodeRabbit support.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 0%
with 26 lines
in your changes are missing coverage. Please review.
Project coverage is 16.01%. Comparing base (
2753171
) to head (fd50a4e
). Report is 60 commits behind head on develop.:exclamation: Current head fd50a4e differs from pull request most recent head e365237
Please upload reports for the commit e365237 to get more accurate results.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary by CodeRabbit
New Features
Bug Fixes
Refactor
Chores