-
Notifications
You must be signed in to change notification settings - Fork 131
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
Clone Git monorepos with several dependencies only once #1275
Conversation
deku: | ||
git: https://github.com/mikesol/purescript-deku.git | ||
subdir: deku | ||
ref: 276f48adde3d9354f61917f7e9ae2ae7b43df6b2 | ||
deku-core: | ||
git: https://github.com/mikesol/purescript-deku.git | ||
subdir: deku-core | ||
ref: 98c67533cc8c399aa643b495d3c02bab963e5b80 | ||
deku-css: | ||
git: https://github.com/mikesol/purescript-deku.git | ||
subdir: deku-css | ||
ref: 4e68a5cec10c91aa3377ce69cc97c276936a1194 | ||
deku-dom: | ||
git: https://github.com/mikesol/purescript-deku.git | ||
subdir: deku-dom | ||
ref: 6b7c392da7782fe0f2e34811e36b11e630e10b26 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I used Mike's example from #1208, but with specific SHAs. This should be fine until the repo is deleted or its history is rewritten.
8969f96
to
ade6fee
Compare
ce182d4
to
0b2193a
Compare
core/src/FS.purs
Outdated
@@ -15,6 +15,7 @@ module Spago.FS | |||
, readTextFile | |||
, readYamlDocFile | |||
, readYamlFile | |||
, rmRf |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wouldn't like this in application code - it looks like it's only used for tests so we should move it into the Test Prelude or just in the Test.Spago.Build.Monorepo
module
98e384a
to
1c71e40
Compare
f7df94a
to
9c4bcde
Compare
@fsoikin could I ask you to not force-push? Seeing all the commits is actually useful for reviewing things, and we squash-merge all the PRs so there's no effect on what ends up in trunk. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing patch, great work 💯
Thank you, we have tested this and it works great! |
Description of the change
Fixed #1208
.spago/g/
.Checklist:
[ ] Added some example of the new feature to theREADME