-
Notifications
You must be signed in to change notification settings - Fork 21
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
Tests to run on .github folder update, to verify that all essential scripts are working correctly #285
Conversation
Treating pr.yaml as a Workflow definition |
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.
а зачем коммитить все эти файлы? (*testing_out_stuff*)
Это тестовые данные на которых быстро можно прогнать скрипты и сравнить полученный результат с каноничным. |
по моему это плохая идея во первых там куча рандомных данных типа |
Их не надо перезаливать, это статические данные, которые я специально обрезал, чтобы они занимали мало места, они никак не завязаны на сборку которая уже есть. Основное занимаемое место с ними просто в inode-ах. |
Я могу это всё в архив положить, если прям кому-то сильно мешает. Но если что, сорцы у нас намного более часто меняются и они занимают существенно больше места. |
Если эти данные никогда не будут менятса можно действительно как к бинарнику относится залить на s3 zip скачивать и прогонять тесты. Можно ещё git lfs заюзать и хранить ссылку в repository. Обычно так делают потому что если заливаеш новую версию бинарника в git истории остаётся старая версия а diff между бинарниками не всегда маленький |
…cripts are working correctly
25b7bed
to
021e1db
Compare
Treating build_and_test_on_demand_cmake.yaml as a Workflow definition |
Унёс структуру директорий на s3 |
No description provided.