-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add planetscale/vitess
module
#365
Conversation
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.
Even though the source repo already has a buf.yaml
, we're adding one here to control the destination ourselves. See related comment #344 (comment)
@@ -0,0 +1 @@ | |||
770e14433716a816905f267650191319f7d0515b |
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.
Latest git main
hash at the moment of opening this PR.
Successfully tested (includes LICENSE contents) in #366, see comments there. |
|
||
# Copy only curated subset of files from that repo into a tmp module dir using `rsync.incl` file | ||
# on each module's static dir | ||
--include-from="${module_static_path}/rsync.incl" |
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.
Nice - this is helpful to see it expanded like this.
Re-tested again: #371 |
References #360