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

README: Update and add tips about usage in Docker and CI #208

Merged
merged 2 commits into from
Feb 20, 2020

Conversation

harriha
Copy link
Contributor

@harriha harriha commented Feb 19, 2020

Ran into some trouble in my Docker-based setup also on Travis, thus came up with some suggestions. Feel free to modify if need be.

The note referring to #185 wasn't about Gitlab specifically, ran into the same issue on local Docker builds as well as on Docker @ Travis, thus generalized that note a bit.

building in Docker, you might need to adjust configuration in `.npmrc`. See
[#185](https://github.com/ds300/patch-package/issues/185).
- In your `Dockerfile`, remember to copy over the patch files *before* running
`[npm|yarn] install`
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To clarify: this is pretty obvious once you realize it, but in addition to myself, others have apparently tripped over this as well, so thought that perhaps worth a mention.

Copy link
Owner

@ds300 ds300 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great, thank you! 🙏

@ds300 ds300 merged commit a99fe0f into ds300:master Feb 20, 2020
@vlrevolution
Copy link

Doesn't seem to work for me in 2024. Can you share your Dockerfile in full please?

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.

3 participants