Open jodobear opened 4 years ago
@Fonta1n3 @shannona Can you review this PR early next week?
After this settles, my next question is going to be: how can we merge this into the StackScript for Linode, since that's our preferred methodology for the Learning Bitcoin course (and at the least we'd like the automatic c-lightning install.)
After this settles, my next question is going to be: how can we merge this into the StackScript for Linode, since that's our preferred methodology for the Learning Bitcoin course (and at the least we'd like the automatic c-lightning install.)
I can just merge relevant parts into a single script with c-lightning for the StackScript. Does that work?
After this settles, my next question is going to be: how can we merge this into the StackScript for Linode, since that's our preferred methodology for the Learning Bitcoin course (and at the least we'd like the automatic c-lightning install.)
I can just merge relevant parts into a single script with c-lightning for the StackScript. Does that work?
Definitely!
@jodobear wrote
I can just merge relevant parts into a single script with c-lightning for the StackScript. Does that work?
Did this happen? Should this PR be closed, or is there still useful code for the future here?
@icculp @nochiel — any desire to pick up this PR, test it, and see if we can't merged into master?
@icculp @nochiel — any desire to pick up this PR, test it, and see if we can't merged into master?
I've begun reviewing the PR.
Funny timing Nick, I started looking at it just a little while ago as well
@shannona is your review at https://github.com/BlockchainCommons/Bitcoin-Standup-Scripts/pull/15/#pullrequestreview-503417628 now resolved?
Installs: