cloudinary / cloudinary/cloudinary-react-native

Issue: Improve `bootstrap.js` to support both Yarn and NPM with clearer errors

未关闭
#50 3 条评论 0 个 reaction 已指派 1 人 已被 @adimiz1 认领 在 GitHub 查看
主要语言
TypeScript
星标
28
派生
21
PR 合并指标
30 天内没有已合并 PR

描述

## Feature request for Cloudinary React Native SDK

Improve `scripts/bootstrap.js` to support both Yarn and NPM with clearer error handling.

## Explain your use case

When contributing to this repository, new developers typically start with `npm install`.
However, the current `scripts/bootstrap.js` assumes Yarn is installed.
If Yarn is missing, the script fails without guidance, making onboarding harder for contributors who default to npm.

## Describe the problem you’re trying to solve

Right now:
- The script hardcodes Yarn and fails if it’s not installed.
- There is no fallback to `npm run bootstrap`.
- If `bootstrap` is not defined in `package.json`, the script exits with no explanation.

This creates friction for new contributors and may result in unnecessary issues being opened.

## Do you have a proposed solution?

Yes.
- Detect if Yarn is available; if not, fall back to `npm run bootstrap`.
- Allow contributors to explicitly prefer npm (e.g. with a flag like `--use-npm`).
- Validate that a `bootstrap` script exists in `package.json` before trying to run it.
- Print clear, contributor-friendly error messages when something fails.

This change would not alter the default behavior for Yarn users, but it would improve reliability, reduce confusion, and make onboarding easier for developers using npm.

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。