-
Notifications
You must be signed in to change notification settings - Fork 3
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
2785 Updated the dependencies to the latest version #48
Closed
Closed
Changes from all commits
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
8aa6072
2785 Updated the dependencies to the latest version
bkarambe c30b6b3
2785 Reverted the node engine unsupported deps
bkarambe 2fa43a9
2785 Updated typescript-eslint deps
bkarambe 4d39a11
Updated node version in config yml
bkarambe 7f35411
Regenrated yarn lock files
bkarambe 425ed98
2785 Added a typecasting for failing lint
bkarambe ebb6afb
2785 Updated blank project dependencies
bkarambe 7476df0
Updated authentication dependencies
bkarambe a8743fe
Updated routing dependencies
bkarambe 482d2c3
Updated dependencies
bkarambe b4f1e61
Updated dependencies in root package
bkarambe File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,31 +1,31 @@ | ||
{ | ||
"dependencies": [ | ||
"@brightlayer-ui/colors@^3.0.1", | ||
"@brightlayer-ui/icons-svg@^1.8.0", | ||
"@brightlayer-ui/icons-svg@^1.8.1", | ||
"@brightlayer-ui/react-native-auth-workflow@^4.0.0", | ||
"@brightlayer-ui/react-native-components@^6.0.1", | ||
"@brightlayer-ui/react-native-components@^6.0.2", | ||
"@brightlayer-ui/react-native-themes@^6.0.0", | ||
"@brightlayer-ui/react-native-vector-icons@^1.4.0", | ||
"@react-native-async-storage/async-storage@^1.14.1", | ||
"@react-native-community/masked-view@^0.1.10", | ||
"@react-navigation/drawer@^5.12.3", | ||
"@react-navigation/native@^5.9.3", | ||
"@react-navigation/stack@^5.14.3", | ||
"date-fns@^2.19.0", | ||
"i18next@^20.0.0", | ||
"react-i18next@^11.8.7", | ||
"react-native-gesture-handler@^1.10.3", | ||
"react-native-keyboard-aware-scroll-view@^0.9.3", | ||
"react-native-modal@^12.0.2", | ||
"react-native-pager-view@^5.0.0", | ||
"react-native-paper@^4.7.2", | ||
"react-native-reanimated@^1.0.0", | ||
"react-native-safe-area-context@^3.2.0", | ||
"react-native-screens@^3.0.0", | ||
"react-native-svg@^12.1.0", | ||
"react-native-svg-transformer@^0.14.3", | ||
"react-native-vector-icons@^8.1.0", | ||
"react-native-webview@^10.3.2" | ||
"@brightlayer-ui/react-native-vector-icons@^1.4.1", | ||
"@react-native-async-storage/async-storage@^1.15.14", | ||
"@react-native-community/masked-view@^0.1.11", | ||
"@react-navigation/drawer@^6.1.8", | ||
"@react-navigation/native@^6.0.6", | ||
"@react-navigation/stack@^6.0.11", | ||
"date-fns@^2.28.0", | ||
"i18next@^21.6.5", | ||
"react-i18next@^11.15.3", | ||
"react-native-gesture-handler@^2.1.0", | ||
"react-native-keyboard-aware-scroll-view@^0.9.5", | ||
"react-native-modal@^13.0.0", | ||
"react-native-pager-view@^5.4.9", | ||
"react-native-paper@^4.11.1", | ||
"react-native-reanimated@^2.3.1", | ||
"react-native-safe-area-context@^3.3.2", | ||
"react-native-screens@^3.10.1", | ||
"react-native-svg@^12.1.1", | ||
"react-native-svg-transformer@^1.0.0", | ||
"react-native-vector-icons@^9.0.0", | ||
"react-native-webview@^11.15.1" | ||
], | ||
"devDependencies": ["@types/react-native-vector-icons", "jest-transform-stub"] | ||
} |
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,31 +1,31 @@ | ||
{ | ||
"dependencies": [ | ||
"@brightlayer-ui/colors@^3.0.1", | ||
"@brightlayer-ui/icons-svg@^1.8.0", | ||
"@brightlayer-ui/icons-svg@^1.8.1", | ||
"@brightlayer-ui/react-native-auth-workflow@^4.0.0", | ||
"@brightlayer-ui/react-native-components@^6.0.1", | ||
"@brightlayer-ui/react-native-components@^6.0.2", | ||
"@brightlayer-ui/react-native-themes@^6.0.0", | ||
"@brightlayer-ui/react-native-vector-icons@^1.4.0", | ||
"@react-native-async-storage/async-storage@^1.14.1", | ||
"@react-native-community/masked-view@^0.1.10", | ||
"@react-navigation/drawer@^5.12.3", | ||
"@react-navigation/native@^5.9.3", | ||
"@react-navigation/stack@^5.14.3", | ||
"date-fns@^2.19.0", | ||
"i18next@^20.0.0", | ||
"react-i18next@^11.8.7", | ||
"react-native-gesture-handler@^1.10.3", | ||
"react-native-keyboard-aware-scroll-view@^0.9.3", | ||
"react-native-modal@^12.0.2", | ||
"react-native-pager-view@^5.0.0", | ||
"react-native-paper@^4.7.2", | ||
"react-native-reanimated@^1.0.0", | ||
"react-native-safe-area-context@^3.2.0", | ||
"react-native-screens@^3.0.0", | ||
"react-native-svg@^12.1.0", | ||
"react-native-svg-transformer@^0.14.3", | ||
"react-native-vector-icons@^8.1.0", | ||
"react-native-webview@^10.3.2" | ||
"@brightlayer-ui/react-native-vector-icons@^1.4.1", | ||
"@react-native-async-storage/async-storage@^1.15.14", | ||
"@react-native-community/masked-view@^0.1.11", | ||
"@react-navigation/drawer@^6.1.8", | ||
"@react-navigation/native@^6.0.6", | ||
"@react-navigation/stack@^6.0.11", | ||
"date-fns@^2.28.0", | ||
"i18next@^21.6.5", | ||
"react-i18next@^11.15.3", | ||
"react-native-gesture-handler@^2.1.0", | ||
"react-native-keyboard-aware-scroll-view@^0.9.5", | ||
"react-native-modal@^13.0.0", | ||
"react-native-pager-view@^5.4.9", | ||
"react-native-paper@^4.11.1", | ||
"react-native-reanimated@^2.3.1", | ||
"react-native-safe-area-context@^3.3.2", | ||
"react-native-screens@^3.10.1", | ||
"react-native-svg@^12.1.1", | ||
"react-native-svg-transformer@^1.0.0", | ||
"react-native-vector-icons@^9.0.0", | ||
"react-native-webview@^11.15.1" | ||
], | ||
"devDependencies": ["jest-transform-stub"] | ||
} |
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
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.
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.
We shouldn't be using the
any
typeThere 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.
This is a little tricky;
getInitialState
is the correct type already (even without theany
cast) but the auth-ts template fails linting because it doesn't have access to the type definitions found within@react-navigation/native
.I'm confused as to why we're creating templates without having the required deps installed at the root level.
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 think this story is going to evolve into something larger than expected; how is linting working without the type dependencies installed?
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.
@daileytj do you have thoughts on this one? We might need to update the root package.json so it contains all the dependencies that each template requires; this is how the angular one does it currently.
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.
The root package.json absolutely should contain every dependency that is in any template
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.
This is how it was initially written so that the demo project could run every template locally for testing purposes