maniac-tech / maniac-tech/react-native-expo-read-sms

Add npm version in engines in package.json

Open
#25 0 comments 0 reactions 1 assignee View on GitHub

@maniac-tech is already working on this.

Since Feb 21, 2023.

enhancement good first issue WIP
Dominant language
Java
Stars
85
Forks
32
PR merge metrics
No merged PRs in 30d

Description

The error log shared in the original description mentions that there is a conflict of dependencies.

A simple way to resolve this temporarily is to use the attribute `--force`or `--legacy-peer-deps`along with `npm install`
The final command will look like: `npm install --force`/ `npm install --legacy-peer-deps`

Engineers are expected to install peer dependencies manually, this change was introduced in v3.x, and has been updated in v7.x of npm.

You may use the temporary solution for now, till I raise a PR to start using v7.x and above.

_Originally posted by @maniac-tech in https://github.com/maniac-tech/react-native-expo-read-sms/issues/19#issuecomment-1439023747_

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.