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

Refine - remove 'native' from package.json #1931

Closed
wants to merge 1 commit into from

Conversation

jake-nz
Copy link
Contributor

@jake-nz jake-nz commented Aug 3, 2022

There is no native folder in the release but it was referred to in the package file, which causes errors for react-native.

There is no `native` folder in the release but it was referred to in the package file, which causes errors for react-native.
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 3, 2022
@facebook-github-bot
Copy link
Contributor

@drarmstr has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@drarmstr drarmstr added the Component: refine related to refine label Aug 9, 2022
@jake-nz
Copy link
Contributor Author

jake-nz commented Aug 12, 2022

@drarmstr has imported this pull request.

What has happened here? Does that mean it's being accepted by some process outside Github?

@drarmstr
Copy link
Contributor

Yes, it was imported to our internal source repository which is the source of truth for the code base. When changes land in that repository then they are replicated back out to the GitHub repository.

@drarmstr drarmstr mentioned this pull request Aug 16, 2022
drarmstr added a commit to drarmstr/Recoil that referenced this pull request Aug 16, 2022
Summary:
Refine 0.1.1 release:

- Remove reference to `native` directory in `package.json` to cleanup errors for `react-native` (facebookexperimental#1931)
- Export Path class for custom checkers (facebookexperimental#1950, facebookexperimental#1956)

Reviewed By: bsouthga

Differential Revision: D38764896

fbshipit-source-id: f8d6c35fc5c92c285182586e3b725d03c65bcf59
drarmstr added a commit to drarmstr/Recoil that referenced this pull request Aug 17, 2022
Summary:
Pull Request resolved: facebookexperimental#1959

Refine 0.1.1 release:

- Remove reference to `native` directory in `package.json` to cleanup errors for `react-native` (facebookexperimental#1931)
- Export Path class for custom checkers (facebookexperimental#1950, facebookexperimental#1956)

Reviewed By: bsouthga

Differential Revision: D38764896

fbshipit-source-id: dedf75327d5b4103f92ab421240704f73786e742
facebook-github-bot pushed a commit that referenced this pull request Aug 18, 2022
Summary:
Pull Request resolved: #1959

Refine 0.1.1 release:

- Rename `boolean()` export to `bool()` since `boolean` is a reserved word (#1922, #1962)
- Remove reference to `native` directory in `package.json` to cleanup errors for `react-native`. (#1931)
- Export `Path` class for custom checkers. (#1950, #1956)
- Extend the failure message of `union()` and `or()` with each type. (#1961)

Reviewed By: bsouthga

Differential Revision: D38764896

fbshipit-source-id: 5eb0327ab9034f3f0802796d986fd6e7df28f05c
snipershooter0701 pushed a commit to snipershooter0701/Recoil that referenced this pull request Mar 5, 2023
Summary:
There is no `native` folder in the release but it was referred to in the package file, which causes errors for react-native.

Pull Request resolved: facebookexperimental/Recoil#1931

Reviewed By: mondaychen

Differential Revision: D38526707

Pulled By: drarmstr

fbshipit-source-id: b18baeac1f9d1a68c404b7eba3c2aaf46c005359
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build / infra CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Component: refine related to refine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants