Skip to content

Put back the README.md file in the npm package#6256

Merged
piaskowyk merged 1 commit intomainfrom
@piaskowyk/restore-readme
Jul 11, 2024
Merged

Put back the README.md file in the npm package#6256
piaskowyk merged 1 commit intomainfrom
@piaskowyk/restore-readme

Conversation

@piaskowyk
Copy link
Copy Markdown
Member

Summary

This PR adds the README.md file back to the NPM package. After migration to a monorepo, the README.md file's location was changed to the parent directory. Unfortunately, you can't include a file from the parent directory in an npm package, and symlinks don't work for this either.

Test plan

run npm pack and check if README.md is inside

@piaskowyk piaskowyk requested review from szydlovsky and tomekzaw July 11, 2024 08:11
@tomekzaw tomekzaw changed the title Put back the readme.md file in the NPM package. Put back the README.md file in the npm package Jul 11, 2024
@piaskowyk piaskowyk added this pull request to the merge queue Jul 11, 2024
Merged via the queue into main with commit 7301438 Jul 11, 2024
@piaskowyk piaskowyk deleted the @piaskowyk/restore-readme branch July 11, 2024 08:30
r0h0gg6 pushed a commit to r0h0gg6/react-native-reanimated that referenced this pull request Jul 28, 2025
## Summary

This PR adds the `README.md` file back to the NPM package. After
migration to a monorepo, the `README.md` file's location was changed to
the parent directory. Unfortunately, you can't include a file from the
parent directory in an npm package, and symlinks don't work for this
either.

## Test plan

run `npm pack` and check if README.md is inside
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants