chore: include apps directory in format-js script#4082
Merged
Conversation
sgaczol
requested review from
Copilot,
kkafar,
kligarski,
kmichalikk and
t0maboro
Collaborator
Author
|
I guess we should do the same thing with |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s JS/TS formatting workflow by including the apps directory in the format-js script, and applies Prettier-driven formatting updates across apps/ sources (tests, shared utilities, and gamma containers).
Changes:
- Expand
format-jsinpackage.jsonto format files underapps/. - Apply formatting updates across
apps/(imports, wrapping, trailing commas, semicolons, JSX formatting).
Reviewed changes
Copilot reviewed 94 out of 97 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Include apps in format-js Prettier glob. |
| apps/src/tests/single-feature-tests/tabs/test-tabs-tab-bar-experimental-user-interface-style-ios/ThemeScreen.tsx | Prettier formatting. |
| apps/src/tests/single-feature-tests/tabs/test-tabs-tab-bar-experimental-user-interface-style-ios/index.tsx | Prettier formatting (imports/strings wrapping). |
| apps/src/tests/single-feature-tests/tabs/test-tabs-stale-update-rejection/index.tsx | Prettier formatting (object literals/whitespace). |
| apps/src/tests/single-feature-tests/tabs/test-tabs-special-effects-scroll-to-top/scenario-description.ts | Prettier formatting. |
| apps/src/tests/single-feature-tests/tabs/test-tabs-special-effects-scroll-to-top/index.tsx | Prettier formatting (JSX/object literals). |
| apps/src/tests/single-feature-tests/tabs/test-tabs-prevent-native-selection/index.tsx | Prettier formatting (JSX props/quotes). |
| apps/src/tests/single-feature-tests/tabs/test-tabs-override-scroll-view-content-inset-ios/index.tsx | Prettier formatting (JSX simplification). |
| apps/src/tests/single-feature-tests/tabs/test-tabs-more-navigation-controller-ios/index.tsx | Prettier formatting (imports/object literals). |
| apps/src/tests/single-feature-tests/tabs/test-tabs-lifecycle-events/scenario-description.ts | Prettier formatting. |
| apps/src/tests/single-feature-tests/tabs/test-tabs-lifecycle-events/index.tsx | Prettier formatting (JSX). |
| apps/src/tests/single-feature-tests/tabs/test-tabs-bottom-accessory-layout-ios/index.tsx | Prettier formatting (JSX wrapping/quotes). |
| apps/src/tests/single-feature-tests/tabs/test-tabs-appearance-defined-by-selected-tab/index.tsx | Prettier formatting (imports/JSX). |
| apps/src/tests/single-feature-tests/form-sheet/test-form-sheet-preferred-corner-radius-ios/index.tsx | Prettier formatting (imports/state init/JSX). |
| apps/src/tests/single-feature-tests/form-sheet/test-form-sheet-largest-undimmed-detent-index-ios/index.tsx | Prettier formatting (imports/state init). |
| apps/src/tests/single-feature-tests/form-sheet/test-form-sheet-initial-detent-index-ios/index.tsx | Prettier formatting (imports). |
| apps/src/tests/issue-tests/TestSplit/SplitWithNativeStack/SplitWithNativeStackSheet.tsx | Prettier formatting (props/options). |
| apps/src/tests/issue-tests/TestSplit/SplitWithNativeStack/SplitWithNativeStackBase.tsx | Prettier formatting (props/exports). |
| apps/src/tests/issue-tests/TestSplit/SplitWithNativeStack/index.tsx | Prettier formatting (imports/exports). |
| apps/src/tests/issue-tests/TestSplit/index.tsx | Prettier formatting (imports/return). |
| apps/src/tests/issue-tests/TestSplit/helpers/index.tsx | Prettier formatting (semicolon). |
| apps/src/tests/issue-tests/TestScreenAnimationV5.tsx | Prettier formatting (import semicolon). |
| apps/src/tests/issue-tests/TestScreenAnimation.tsx | Prettier formatting (import semicolon). |
| apps/src/tests/issue-tests/TestSafeAreaViewIOS/stack-v4/TestScreen.tsx | Prettier formatting (object formatting). |
| apps/src/tests/issue-tests/TestSafeAreaViewIOS/split-view/index.tsx | Prettier formatting (import). |
| apps/src/tests/issue-tests/TestMemoryLeak.tsx | Prettier formatting (imports/JSX wrapping). |
| apps/src/tests/issue-tests/TestFormSheet.tsx | Prettier formatting (imports/types/JSX wrapping). |
| apps/src/tests/issue-tests/TestBottomTabs/tabs/Tab3.tsx | Prettier formatting (JSX wrapping). |
| apps/src/tests/issue-tests/TestAnimation.tsx | Prettier formatting (types/JSX wrapping). |
| apps/src/tests/issue-tests/Test649.js | Prettier formatting (JSX wrapping). |
| apps/src/tests/issue-tests/Test645.js | Prettier formatting (JSX wrapping). |
| apps/src/tests/issue-tests/Test556.tsx | Prettier formatting (spacing/trailing commas). |
| apps/src/tests/issue-tests/Test3369.tsx | Prettier formatting (type spacing). |
| apps/src/tests/issue-tests/Test3345.tsx | Prettier formatting (comments/JSX wrapping). |
| apps/src/tests/issue-tests/Test3282.tsx | Prettier formatting (imports/types). |
| apps/src/tests/issue-tests/Test3271.tsx | Prettier formatting (types/JSX wrapping). |
| apps/src/tests/issue-tests/Test3265.tsx | Prettier formatting (types/JSX wrapping). |
| apps/src/tests/issue-tests/Test3239.tsx | Prettier formatting (imports/types/log formatting). |
| apps/src/tests/issue-tests/Test3212/StackScenario.tsx | Prettier formatting; React import issue noted in review comments. |
| apps/src/tests/issue-tests/Test3212/StackInStackScenario.tsx | Prettier formatting (imports/JSX wrapping). |
| apps/src/tests/issue-tests/Test3212/ScrollViewTemplate.tsx | Prettier formatting (imports/JSX wrapping). |
| apps/src/tests/issue-tests/Test3212/context.tsx | Prettier formatting (types/object spacing). |
| apps/src/tests/issue-tests/Test3212/Config.tsx | Prettier formatting; React import issue noted in review comments. |
| apps/src/tests/issue-tests/Test3045.tsx | Prettier formatting (imports/JSX/options). |
| apps/src/tests/issue-tests/Test2933.tsx | Prettier formatting (imports/semicolons/JSX wrapping). |
| apps/src/tests/issue-tests/Test2877.tsx | Prettier formatting (JSX wrapping). |
| apps/src/tests/issue-tests/Test2842.tsx | Prettier formatting (imports/JSX wrapping). |
| apps/src/tests/issue-tests/Test2819.tsx | Prettier formatting (imports). |
| apps/src/tests/issue-tests/Test2811.tsx | Prettier formatting (imports/JSX wrapping). |
| apps/src/tests/issue-tests/Test2809/Shared.tsx | Prettier formatting (JSX wrapping/function formatting). |
| apps/src/tests/issue-tests/Test2809/index.tsx | Prettier formatting (const wrapping/JSX wrapping). |
| apps/src/tests/issue-tests/Test2789.tsx | Prettier formatting (types/JSX wrapping). |
| apps/src/tests/issue-tests/Test2767.tsx | Prettier formatting (types/JSX wrapping). |
| apps/src/tests/issue-tests/Test2717.tsx | Prettier formatting (types/JSX wrapping). |
| apps/src/tests/issue-tests/Test2714.tsx | Prettier formatting (JSX wrapping). |
| apps/src/tests/issue-tests/Test2668.tsx | Prettier formatting (types/JSX wrapping). |
| apps/src/tests/issue-tests/Test2631.tsx | Prettier formatting (imports/JSX wrapping). |
| apps/src/tests/issue-tests/Test2552.tsx | Prettier formatting (imports/JSX wrapping). |
| apps/src/tests/issue-tests/Test2538.tsx | Prettier formatting (imports/types). |
| apps/src/tests/issue-tests/Test2466.tsx | Prettier formatting (imports/types/log formatting). |
| apps/src/tests/issue-tests/Test2332.tsx | Prettier formatting (object spacing). |
| apps/src/tests/issue-tests/Test2320.tsx | Prettier formatting (imports/JSX/options). |
| apps/src/tests/issue-tests/Test2235.tsx | Prettier formatting (JSX wrapping/semicolons). |
| apps/src/tests/issue-tests/Test2232.tsx | Prettier formatting (trailing newline removal). |
| apps/src/tests/issue-tests/Test2175.tsx | Prettier formatting (JSX wrapping/quotes). |
| apps/src/tests/issue-tests/Test2167.tsx | Prettier formatting (imports/JSX wrapping). |
| apps/src/tests/issue-tests/Test1649/types.tsx | Prettier formatting (types/imports). |
| apps/src/tests/issue-tests/Test1649/state.tsx | Prettier formatting (atoms wrapping). |
| apps/src/tests/issue-tests/Test1649/screens/Third.tsx | Prettier formatting (imports/quotes). |
| apps/src/tests/issue-tests/Test1649/screens/SheetScreenWithTextInput.tsx | Prettier formatting (imports). |
| apps/src/tests/issue-tests/Test1649/screens/SheetScreenWithScrollView.tsx | Prettier formatting (imports). |
| apps/src/tests/issue-tests/Test1649/screens/SheetScreen.tsx | Prettier formatting (imports/listener callback). |
| apps/src/tests/issue-tests/Test1649/screens/Second.tsx | Prettier formatting (imports/JSX/quotes). |
| apps/src/tests/issue-tests/Test1649/screens/PushWithScrollView.tsx | Prettier formatting (imports/function params). |
| apps/src/tests/issue-tests/Test1649/screens/ModalScreen.tsx | Prettier formatting (imports). |
| apps/src/tests/issue-tests/Test1649/screens/Home.tsx | Prettier formatting (imports). |
| apps/src/tests/issue-tests/Test1649/routes.tsx | Prettier formatting (types/quotes). |
| apps/src/tests/issue-tests/Test1649/components/GestureHandlerButton.tsx | Prettier formatting (imports/callback/style object). |
| apps/src/tests/issue-tests/Test1649/components/Footer.tsx | Prettier formatting (imports). |
| apps/src/tests/issue-tests/Test1649/components/CommonSheetContent.tsx | Prettier formatting (imports/JSX/quotes). |
| apps/src/shared/utils/deep-merge.ts | Prettier formatting (trailing newline). |
| apps/src/shared/Spacer.tsx | Prettier formatting (semicolon). |
| apps/src/shared/Rectangle.tsx | Prettier formatting (types/JSX wrapping). |
| apps/src/shared/gamma/containers/tabs/TabsContainerWithHostConfigContext.tsx | Prettier formatting (destructuring). |
| apps/src/shared/gamma/containers/tabs/TabsContainerItem.tsx | Prettier formatting (imports/template literal/hooks). |
| apps/src/shared/gamma/containers/tabs/TabsContainer.types.tsx | Prettier formatting (type formatting). |
| apps/src/shared/gamma/containers/tabs/TabsContainer.tsx | Prettier formatting (destructuring/hooks). |
| apps/src/shared/gamma/containers/tabs/reducer.tsx | Prettier formatting (conditional wrapping). |
| apps/src/shared/gamma/containers/tabs/presets.ts | Prettier formatting (type wrapping). |
| apps/src/shared/gamma/containers/tabs/index.ts | Prettier formatting (export type formatting). |
| apps/src/shared/gamma/containers/stack/StackContainer.tsx | Prettier formatting (param destructuring). |
| apps/src/shared/gamma/containers/stack/reducer.tsx | Prettier formatting (line wrapping). |
| apps/src/shared/gamma/containers/shared/id-generator.ts | Prettier formatting (trailing newline). |
| apps/src/screens/SwipeBackAnimation.tsx | Prettier formatting (generic type wrapping/JSX). |
| apps/src/screens/StackPresentation.tsx | Prettier formatting (JSX props wrapping). |
| apps/src/screens/PreventRemove.tsx | Prettier formatting (imports/whitespace). |
| apps/src/screens/BarButtonItems.tsx | Prettier formatting (object literal wrapping). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
1
to
2
| import React, { Button, Text } from 'react-native'; | ||
| import { useScrollEdgeEffectsConfigContext } from './context'; |
Comment on lines
+1
to
+3
| import React from 'react-native'; | ||
| import { createNativeStackNavigator } from "@react-navigation/native-stack"; | ||
| import { ConfigWithNavigation } from "./Config"; | ||
| import { NavigationContainer, NavigationIndependentTree } from "@react-navigation/native"; | ||
| import { createNativeStackNavigator } from '@react-navigation/native-stack'; | ||
| import { ConfigWithNavigation } from './Config'; |
sgaczol
force-pushed
the
@sgaczol/format-apps
branch
from
040d33a to
c1276b7
Compare
sgaczol
force-pushed
the
@sgaczol/format-apps
branch
from
c1276b7 to
a43da36
Compare
kkafar
approved these changes
May 25, 2026
kkafar
left a comment
kkafar
left a comment
Member
There was a problem hiding this comment.
Looks sensible ✅
I believe you haven't made any manual changes to apps/src.
The package.json change seems solid.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
appsdirectory wasn't included informat-jsscript.Closes https://github.com/software-mansion/react-native-screens-labs/issues/785
Changes
package.jsonappsdirectoryBefore & after - visual documentation
N/A
Test plan
N/A
Checklist