-
Notifications
You must be signed in to change notification settings - Fork 32
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
Added button and action on PR #54
Conversation
Added manual run and PR
Added button and action on PR
* Update _config.yml
@dkorzhov а зачем пустой файл |
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.
@dkorzhov Огромное спасибо за эту штуку! Добавь себя пожалуйста в https://github.com/SemyonSinchenko/qmlcourse/blob/master/qmlcourseRU/book/authors.md, там после раздела "Основные ревьюеры" создай раздел "Техническая поддержка" или что-то такое и добавь туда себя.
Оно сейчас падает просто потому, что надо сначала смержить, да? |
это один из вариантов реализации, это разные ветки качаются в локальную папку. |
Сейчас падает, потому что не хватате прав на выполнение в этом репозитории. Надо проверить в настройках разрешения на github actions, в том числе на доступ к чтению и записи прав. Но сначала лучше сделать merge, может это тоже влияет. |
надо будет наверное в .gitignore прописать файлы и папки. Он автоматом создается при билде книги. Это я видимо когда тестировал запись в тот же репозиторий, он туда попал и я его не заметил. |
А ты можешь удалить его из реквеста сейчас? Ну и сразу добавить в gitignore другие артифакты книги. |
Готово, .gitignore пока не трогал, там надо отдельно будет смотреть |
No description provided.