Makefile for os linux and macos #135
Conversation
| # Only run this when the master branch changes | ||
| on: | ||
| workflow_dispatch: |
There was a problem hiding this comment.
это типа только на push или и на pr также будет срабатывать, а то не наблюдаю?
There was a problem hiding this comment.
это типа только на push или и на pr также будет срабатывать, а то не наблюдаю?
не, это полностью ручной запуск.
добавил, чтобы можно было проверять работоспособность на серверах github.
этот код для использования на локальных машинах.
There was a problem hiding this comment.
это типа только на push или и на pr также будет срабатывать, а то не наблюдаю?
не, это полностью ручной запуск.
добавил, чтобы можно было проверять работоспособность на серверах github.
этот код для использования на локальных машинах.
уточню, код не из этого workflow для локальных машин, а из makefile.
этот workflow просто для дополнительной прогонки на серверах, доп. проверка работоспособности кода из makefile. Но даже это не гарантирует, что у пользователей код заработает. Все зависит от ОС и установленных пакетов.
There was a problem hiding this comment.
а почему внизу не видно что этот workflow запустился
There was a problem hiding this comment.
а почему внизу не видно что этот workflow запустился
может не так понял вопрос, но пока этой джобы нет в мастер ветки, мы ее запустить не можем
There was a problem hiding this comment.
увидел Only run this when the master branch changes
на pr в master не будет работать получается, только на push в master?
There was a problem hiding this comment.
ссори за коммент, он не валидный
наде его удалить, пропустил
pr и push работать не будут, только ручной запуск
There was a problem hiding this comment.
а как запустить-то, а то в https://github.com/SemyonSinchenko/qmlcourse/actions не вижу для выбора?
There was a problem hiding this comment.
Это особенность, что запускается только из ветки, которая стоит как default.
Сейчас это master, поэтому надо сначала сделать merge этого PR.
Поэтому мне и удобнее в своем форке тестировать. Там я меняю ветки по умолчанию.
| run: | ||
| shell: bash | ||
| strategy: | ||
| fail-fast: false |
There was a problem hiding this comment.
а на что это влияет?
если false и одна из джоб падает, вторая продолжает выполняться иначе сразу все прерываются
|
|
||
| # This job installs dependencies, build the book, and pushes it to `gh-pages` | ||
| jobs: | ||
| deploy-book: |
There was a problem hiding this comment.
не особо пока догоняю что происходит в этой job-е. особенно что matrix.os определяется после, а не до вызова в runs-on, конечно если это именно оно
There was a problem hiding this comment.
не особо пока догоняю что происходит в этой job-е. особенно что
matrix.osопределяется после, а не до вызова вruns-on, конечно если это именно оно
в данном случае порядок значения не имеет, они находятся на одном уровне
этот участок отвечает за запуск кода на двух ОС, на это влияют настроки параметра strategy
There was a problem hiding this comment.
а зачем тогда для определения ос использовать отдельную job-у, ведь это можно сделать в той, где что-то запускается через strategy после runs-on?
There was a problem hiding this comment.
не совсем понял
у нас есть джоба, которая прогоняет билд на трех ОС с использованием операторов github actions. Ее назначение как я понял, было проверить сборку на трех ОС.
Эту джобу я просто добавил, чтобы технически проверять как локально отработает makefile. Я еще одну планирую добавить для Windows среды. В принципе, я могу и не выносить эти технически джобы.
There was a problem hiding this comment.
все норм, из-за комментариев не увидел расположение правильно быстрым взглядом
|
не хватает ещё конечно полного описания что же в конечном итоге делает makefile и как им пользоваться и где |
Кратко я пока тут пояснил https://github.com/SemyonSinchenko/qmlcourse/issues/113#issuecomment-898873506 |
да, видел, но
|
| @@ -0,0 +1,27 @@ | |||
| name: test-makefile-os-linux | |||
There was a problem hiding this comment.
немножко прикопаюсь к названию, внутри имеем os: [ubuntu-latest, macOS-latest] при этом называем -linux. или так норм?
There was a problem hiding this comment.
я тоже думал, либо длинное название дать -ubuntu-macos, тоже не очень
либо как то обобщенно написать, но unix-like не очень)
поэтому оставил linux, криво, но на работоспособность не влияет)
There was a problem hiding this comment.
лучше длинно, но точно описать что делается, нежели запутать
| # Only run this when the master branch changes | ||
| on: | ||
| workflow_dispatch: |
There was a problem hiding this comment.
а как запустить-то, а то в https://github.com/SemyonSinchenko/qmlcourse/actions не вижу для выбора?
|
тогда льем в master чтоли? |
No description provided.