Created Initialization Script + Documentation#1171
Created Initialization Script + Documentation#1171conzty01 wants to merge 9 commits intoRunestoneInteractive:mainfrom
Conversation
…gitignore to ignore backup env files from the init script.
|
@conzty01 Thanks for your contribution! I'll have a closer look at it early next week. |
|
Trying it out now, just keeping a running list of issues or updates.
I think I need to spin up a test droplet to go further. (which I will do) Happy to answer questions about adding books, as I think the docs there should be up to date. |
…-download implementation.
…due to issues with stdin and pipes.
|
OK, I got to the point of adding a book. There is a subset of books that are automatically added by the initdb command including thinkcspy. Which is what I was trying. But I don't think that the |
|
I checked the database after the failure and That was successful. |
|
So I think the real error is: Maybe adding the book as I showed above would be the way forward rather than doing it interactively. |
…t repeatedly appear when we run docker commands.
|
Thanks for the feedback! I've updated the script which should make it less likely for us to run into weird issues with being interactive. I think this may have been the cause of the issues we were having creating a book. That said, there is still a small roadblock in trying to make this script a one-line install using pipes. The root of the issue stems from the fact that piping the output of curl into bash means that the script cannot read from the keyboard for One workaround to keep this on a single line would be process substitution. With that, the command would end up looking like this: In my opinion, this syntax is a bit unintuitive for those that aren't terminal power-users (myself included) and I wouldn't want to confuse/deter anybody from the automated setup process (though maybe it's a teaching moment?). However, it is just a single line and perhaps the chances are good that a person reading the docs would just copy and paste it in to their terminal anyway. If you are satisfied with this implementation, I can update the docs to reflect the new command and talk a bit more about the various parts of the process. |
|
Yeah, this is nearly identical to the command on |
|
Finished last touches and documentation updates for this PR. Please let me know if there is anything else you'd like included or if anything needs to be updated. |
I created an initialization script that can be used to get the project set up and running locally with an interactive wizard instead of the manual steps outlined in the
running.rstdocumentation. This script has been tested in a Windows WSL environment but has been built to function in UNIX environments. Unfortunately, I don't have the resources to test in Linux or MacOS at the moment, but can work on getting that done if necessary.This script essentially runs through the steps outlined in the
running.rstdocumentation in an interactive way-- prompting the user for paths or values. It detects when the script has been run more than once and backs up any .env files in the root directory and provides options for how to handle an existing database:I'd appreciate some feedback on the portions regarding adding a course as I ran into difficulties even with the manual steps in that area. I'm not sure if the docs are out of date, or I am just not understanding something but I'd be happy to take another look and alter the prompts so that this part of the process makes sense for a new user.
This PR was created with the assistance of GitHub Copilot.