Skip to content
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

fix/nokogiri-install-fail #27

Merged
merged 15 commits into from
Jun 18, 2024
Merged

fix/nokogiri-install-fail #27

merged 15 commits into from
Jun 18, 2024

Conversation

yart
Copy link
Contributor

@yart yart commented Jun 18, 2024

No description provided.

@yart yart requested a review from TorinAsakura June 18, 2024 07:36
@yart yart self-assigned this Jun 18, 2024
TorinAsakura
TorinAsakura previously approved these changes Jun 18, 2024
@yart
Copy link
Contributor Author

yart commented Jun 18, 2024

@TorinAsakura судя по всему, основное назначение этого экшна в том, чтобы проверить, что сайт собирается. Но так как мы исключаем md файлы, из которых собирается сайт, то и проверять, получается, нечего.

Я попробовал подсунуть докер образ в экшн — он теперь не запустился. Возможно, отработали игноры для md.erb. А может быть я неправильно описал это в конфиге.

Плюс эта ошибка на убунте связанная с nokogiri — предлагаю попробовать обновиться на ruby 3.2 или 3.3, по идее, там должны были избавиться от этого бага.

В общем, на текущей версии рубей я в тупике пока что.

@yart
Copy link
Contributor Author

yart commented Jun 18, 2024

Да, похоже, что-то нахомутал с докером — экшн не запускается.

@yart yart requested a review from TorinAsakura June 18, 2024 11:05
@TorinAsakura TorinAsakura merged commit eeea2e6 into master Jun 18, 2024
@TorinAsakura TorinAsakura deleted the fix/nokogiri-install-fail branch June 18, 2024 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants