cloudinary / cloudinary/cloudinary-react-native

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

オープン
#50 コメント 3 件 リアクション 0 件 担当者 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 を短くまとめたダイジェスト。