-
Couldn't load subscription status.
- Fork 3
Improved README.md description #82
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
Conversation
Did I capture the move part OK? I found a few other things to clean up/improve also, hence the PR.
|
OK @Tom-van-Woudenberg I think I am done. I also added some example use cases. |
|
|
||
| In `call-deploy-book.yml` itself you can specify the trigger for this workflow. By default, a push to any branch triggers the workflow. You can limit the branches or subdirectories. | ||
|
|
||
| ## Common Usage Examples |
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.
This part is also covered slightly differnetly by https://teachbooks.io/manual/installation-and-setup/versions_URLs.html. @rlanzafame , what do you think? Should we have it in both places?
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.
Maybe we should move the tips into this deploy-book-workflow README/book-page?
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.
One comment, but ready to merge
Did I capture the move part OK?
I found a few other things to clean up/improve also, hence the PR.