microsoft / microsoft/react-native-windows

package cleanup: Remove internal build logic (/Scripts)

Open
#4,811 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Code Cleanup enhancement nice-to-have Workstream: ES Compliance SFI
Dominant language
C++
Stars
17.3k
Forks
1.2k
Avg merge
1d 13h
Merged PRs (30d)
33

Description

This is one of a set of issues mentioned in PR #4796 to continue to improve the shipping package layout.

This tracks removal of /Scripts. from the files property of /package.json.

It is currently used by deploy.js#291 which is part of what customers call.

The pr #4796 has more details

This issue tracks:

  1. Update deploy.js to not depend on it.
  2. The actual removal.

Contributor guide

Open the contributing guide

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.

Research direction

Read PR #4796 for the package-layout context, then inspect vnext/local-cli/runWindows/utils/deploy.js around line 291 and the /Scripts entry in package.json. Update deploy.js so the customer-facing flow no longer depends on vnext/Scripts, remove /Scripts from package.json's files property, and verify the package layout no longer includes it.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react-native
Domain
build-system, cli
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.