Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[RN] Add support for src directory in react-native #42385

Closed
wants to merge 1 commit into from

Conversation

rubennorte
Copy link
Contributor

Summary:
This adds support for having JS files in a directory within the package. The plan is to have 2 subdirectories there:

  • for private modules, with any nested directories (e.g.: ).
  • for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like (mapped to , or a directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the package will be added soon by huntie.

Changelog: [internal]

Differential Revision: D52875999

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Jan 19, 2024
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 19, 2024
Summary:

This adds support for having JS files in a `src` directory within the `react-native` package. The plan is to have 2 subdirectories there:
* `react-native/src/private` for private modules, with any nested directories (e.g.: `react-native/src/private/dom/nodes/ReadOnlyNode.js`).
* `react-native/src/public` for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like `react-native/View` (mapped to `react-native/src/public/View`, or a `dist` directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the `react-native` package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

@analysis-bot
Copy link

analysis-bot commented Jan 19, 2024

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 16,863,260 +17
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 20,250,565 -10
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: a758694
Branch: main

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 19, 2024
Summary:

This adds support for having JS files in a `src` directory within the `react-native` package. The plan is to have 2 subdirectories there:
* `react-native/src/private` for private modules, with any nested directories (e.g.: `react-native/src/private/dom/nodes/ReadOnlyNode.js`).
* `react-native/src/public` for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like `react-native/View` (mapped to `react-native/src/public/View`, or a `dist` directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the `react-native` package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 19, 2024
Summary:

This adds support for having JS files in a `src` directory within the `react-native` package. The plan is to have 2 subdirectories there:
* `react-native/src/private` for private modules, with any nested directories (e.g.: `react-native/src/private/dom/nodes/ReadOnlyNode.js`).
* `react-native/src/public` for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like `react-native/View` (mapped to `react-native/src/public/View`, or a `dist` directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the `react-native` package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 19, 2024
Summary:

This adds support for having JS files in a `src` directory within the `react-native` package. The plan is to have 2 subdirectories there:
* `react-native/src/private` for private modules, with any nested directories (e.g.: `react-native/src/private/dom/nodes/ReadOnlyNode.js`).
* `react-native/src/public` for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like `react-native/View` (mapped to `react-native/src/public/View`, or a `dist` directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the `react-native` package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 19, 2024
…ok#42385)

Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a  directory within the  package. The plan is to have 2 subdirectories there:
*  for private modules, with any nested directories (e.g.: ).
*  for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like  (mapped to , or a  directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the  package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: f104ca1cfac5821bedb675e00c38601c5ab8871f
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 19, 2024
…ok#42385)

Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a  directory within the  package. The plan is to have 2 subdirectories there:
*  for private modules, with any nested directories (e.g.: ).
*  for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like  (mapped to , or a  directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the  package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: 8e2c490107d7f2837a9d00cb91ae636998a1a484
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 19, 2024
…ok#42385)

Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a  directory within the  package. The plan is to have 2 subdirectories there:
*  for private modules, with any nested directories (e.g.: ).
*  for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like  (mapped to , or a  directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the  package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: 8fe740876285a70c535eecc3c2201142af1c8974
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 19, 2024
…ok#42385)

Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a  directory within the  package. The plan is to have 2 subdirectories there:
*  for private modules, with any nested directories (e.g.: ).
*  for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like  (mapped to , or a  directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the  package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: b7ff8131c29d969f61c71588d200ca9d23e43073
rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 21, 2024
…ok#42385)

Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a  directory within the  package. The plan is to have 2 subdirectories there:
*  for private modules, with any nested directories (e.g.: ).
*  for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like  (mapped to , or a  directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the  package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: b884e657007600e2b35ff0ab641bf1e4123054eb
rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 21, 2024
Summary:

This adds support for having JS files in a `src` directory within the `react-native` package. The plan is to have 2 subdirectories there:
* `react-native/src/private` for private modules, with any nested directories (e.g.: `react-native/src/private/dom/nodes/ReadOnlyNode.js`).
* `react-native/src/public` for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like `react-native/View` (mapped to `react-native/src/public/View`, or a `dist` directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the `react-native` package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 21, 2024
…ok#42385)

Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a  directory within the  package. The plan is to have 2 subdirectories there:
*  for private modules, with any nested directories (e.g.: ).
*  for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like  (mapped to , or a  directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the  package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: 1841d32d419eea6daa66a825a6a58d9db4dd5f95
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 21, 2024
…ok#42385)

Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a  directory within the  package. The plan is to have 2 subdirectories there:
*  for private modules, with any nested directories (e.g.: ).
*  for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like  (mapped to , or a  directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the  package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: f1546c043618106ac065b56aaa7e94118fad993b
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 21, 2024
…ok#42385)

Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a  directory within the  package. The plan is to have 2 subdirectories there:
*  for private modules, with any nested directories (e.g.: ).
*  for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like  (mapped to , or a  directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the  package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: a95c4348c28c9d0c547499ef73141a7972d0269c
@rubennorte rubennorte changed the title '[skip ci] [RN] Add support for src directory in react-native [RN] Add support for src directory in react-native Jan 21, 2024
rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 21, 2024
Summary:

This adds support for having JS files in a `src` directory within the `react-native` package. The plan is to have 2 subdirectories there:
* `react-native/src/private` for private modules, with any nested directories (e.g.: `react-native/src/private/dom/nodes/ReadOnlyNode.js`).
* `react-native/src/public` for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like `react-native/View` (mapped to `react-native/src/public/View`, or a `dist` directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the `react-native` package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 21, 2024
Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a `src` directory within the `react-native` package. The plan is to have 2 subdirectories there:
* `react-native/src/private` for private modules, with any nested directories (e.g.: `react-native/src/private/dom/nodes/ReadOnlyNode.js`).
* `react-native/src/public` for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like `react-native/View` (mapped to `react-native/src/public/View`, or a `dist` directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the `react-native` package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: 9c9bd6c14e8f0e047b4e9dd8aa2d38aa6c1728c9
rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 21, 2024
Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a `src` directory within the `react-native` package. The plan is to have 2 subdirectories there:
* `react-native/src/private` for private modules, with any nested directories (e.g.: `react-native/src/private/dom/nodes/ReadOnlyNode.js`).
* `react-native/src/public` for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like `react-native/View` (mapped to `react-native/src/public/View`, or a `dist` directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the `react-native` package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: 4d1906d20c96b6ec536cccf231d5d2c49b08029c
Summary:

This adds support for having JS files in a `src` directory within the `react-native` package. The plan is to have 2 subdirectories there:
* `react-native/src/private` for private modules, with any nested directories (e.g.: `react-native/src/private/dom/nodes/ReadOnlyNode.js`).
* `react-native/src/public` for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like `react-native/View` (mapped to `react-native/src/public/View`, or a `dist` directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the `react-native` package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D52875999

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 22, 2024
Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a `src` directory within the `react-native` package. The plan is to have 2 subdirectories there:
* `react-native/src/private` for private modules, with any nested directories (e.g.: `react-native/src/private/dom/nodes/ReadOnlyNode.js`).
* `react-native/src/public` for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like `react-native/View` (mapped to `react-native/src/public/View`, or a `dist` directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the `react-native` package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: 425126644258987f58e249b67a7da0c625e48bad
rubennorte added a commit to rubennorte/react-native that referenced this pull request Jan 22, 2024
Summary:
Pull Request resolved: facebook#42385

This adds support for having JS files in a `src` directory within the `react-native` package. The plan is to have 2 subdirectories there:
* `react-native/src/private` for private modules, with any nested directories (e.g.: `react-native/src/private/dom/nodes/ReadOnlyNode.js`).
* `react-native/src/public` for public modules, without nested directories. The plan is that the individual modules created in this directory will be public through the index module or directly via something like `react-native/View` (mapped to `react-native/src/public/View`, or a `dist` directory in the published npm package—details TBD).

The enforcement of private modules being inaccessible from outside the `react-native` package will be added soon by huntie.

Changelog: [internal]

Reviewed By: huntie

Differential Revision: D52875999

fbshipit-source-id: 58ee306f4fafd57ea73e9fd5c5401cce73013a26
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Jan 22, 2024
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 3c2c486.

@facebook-github-bot
Copy link
Contributor

This pull request has been reverted by c92f7e5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner Reverted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants