Skip to content
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

Remove React Native from dependency tree #10795

Closed
2 tasks
dustindclark opened this issue Dec 20, 2022 · 3 comments
Closed
2 tasks

Remove React Native from dependency tree #10795

dustindclark opened this issue Dec 20, 2022 · 3 comments

Comments

@dustindclark
Copy link

Is this related to a new or existing framework?

No response

Is this related to a new or existing API?

No response

Is this related to another service?

No response

Describe the feature you'd like to request

React Native is a massive dependency and is unnecessary for all clients. Remove it from dep tree. Here's the path:

└─┬ @aws-amplify/api-graphql@3.0.7
  └─┬ @aws-amplify/core@5.0.7
    └─┬ @aws-sdk/client-cloudwatch-logs@3.6.1
      └─┬ @aws-sdk/middleware-retry@3.6.1
        └─┬ react-native-get-random-values@1.8.0
          └── react-native@0.70.6

Describe the solution you'd like

React Native is a massive dependency and is unnecessary for all clients. Remove it from dep tree. Here's the path:

└─┬ @aws-amplify/api-graphql@3.0.7
  └─┬ @aws-amplify/core@5.0.7
    └─┬ @aws-sdk/client-cloudwatch-logs@3.6.1
      └─┬ @aws-sdk/middleware-retry@3.6.1
        └─┬ react-native-get-random-values@1.8.0
          └── react-native@0.70.6

Describe alternatives you've considered

React Native is a massive dependency and is unnecessary for all clients. Remove it from dep tree. Here's the path:

└─┬ @aws-amplify/api-graphql@3.0.7
  └─┬ @aws-amplify/core@5.0.7
    └─┬ @aws-sdk/client-cloudwatch-logs@3.6.1
      └─┬ @aws-sdk/middleware-retry@3.6.1
        └─┬ react-native-get-random-values@1.8.0
          └── react-native@0.70.6

Additional context

No response

Is this something that you'd be interested in working on?

  • 👋 I may be able to implement this feature request
  • ⚠️ This feature might incur a breaking change
@dustindclark dustindclark added the pending-triage Issue is pending triage label Dec 20, 2022
@tannerabread
Copy link
Contributor

Hi @dustindclark we are aware of this dependency and are discussing the steps necessary to remove it.

Please follow issue #9119 for updates

@tannerabread tannerabread closed this as not planned Won't fix, can't repro, duplicate, stale Dec 20, 2022
@dustindclark
Copy link
Author

Thanks. Sorry for the dupe. I swear I searched for this :/

@abdallahshaban557
Copy link
Contributor

@dustindclark - don't worry about it! Thank you for reporting this!

@chrisbonifacio chrisbonifacio removed the pending-triage Issue is pending triage label May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants