Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upRewrite data management user guide a bit #1409
Conversation
Is this still active? And does it make sense to look at now? |
@danielskatz I just re-read this text, and I'm happy for it to be merged into master. |
…vement
@benclifford - I've made a bunch of small comments in-line. In addition, I think it would make sense to add a little general text under the staging providers header that explains at that point the difference between the FTP, HTTP, HTTPS: separate task staging and FTP, HTTP, HTTPS: in-task staging sections that follow, something like - if you have this situation, use this part below... |
The rest of this looks good to me |
I've gone ahead and made the changes that are small. |
@danielskatz why don't we merge this now. Since you've already done a fair bit of testing with staging, can you try re-reading some of this and recommend changes in a separate PR? |
I suggest we just merge this PR for now - It already has a bunch of changes I added a few weeks ago, and I don't have a good suggestion for the part that I thought still needed something more. |
benclifford commentedOct 28, 2019
putting this up for people to look at, but I want to work on it more