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

Add npm version in engines in package.json

Open

#25 opened on Feb 21, 2023

View on GitHub
 (0 comments) (0 reactions) (1 assignee)Java (32 forks)auto 404
WIPenhancementgood first issue

Repository metrics

Stars
 (84 stars)
PR merge metrics
 (PR metrics pending)

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 --forceor --legacy-peer-depsalong 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

Research direction
Edit the package.json file to add an 'engines' field specifying a minimum npm version. For example: "engines": { "npm": ">=7.0.0" }. Then test with npm install.
Tech stack
javascriptnodejs
Domain
build system
Issue type
Chore
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Git
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
85