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

Attempt to fix book generating #206

Merged
merged 6 commits into from
Oct 20, 2024
Merged

Conversation

burlakvo
Copy link
Contributor

Related issue: #196

It looks like generating fixed, but there are still some warnings regarding wrong anchors:
asciidoctor: WARNING: progit.asc: invalid reference to unknown anchor: ch03-git-branching#ch03-git-branching etc.

I didn't catch why it happens and what differs from the English version

@burlakvo
Copy link
Contributor Author

burlakvo commented Oct 19, 2024

I ran workflow on my fork to retest it, and the build was finished successfully
https://github.com/burlakvo/progit2-uk/actions/runs/11418549954/job/31772123231

However, there are a lot of issues with anchors. Despite those issues, we could generate books

@burlakvo burlakvo marked this pull request as ready for review October 19, 2024 16:54
@hedrok
Copy link
Member

hedrok commented Oct 19, 2024

То заливати? Права тобі просити? У цьому репозиторії точно можна українською, бо кому він ще треба, крім тих, хто розуміє українську? :)

@burlakvo
Copy link
Contributor Author

Так, можна зливати. На помилки тоді відкрию завдання (якось згрупую їх і в окремих завданнях опишу чи всі в одне, а там видно буде)

Щодо прав. Мені цікаво зайнятися цим проєктом та я не певен, що матиму достатньо часу. Наприклад, зараз у мене міні відпустка і я мав можливість декілька днів поспіль по декілька годин цим займатися. Надалі моя активність знизиться. Якщо так підходить і надалі будуть бодай мінімальні рев'ю - то я готовий :)

Та я на роботі звик, що всі запити на злиття англійською, то думка така може й була, але і тут собі англійською написав

@hedrok
Copy link
Member

hedrok commented Oct 20, 2024

Якщо тобі все одно треба мінімальні ревʼю, то почнімо поки без прав - буду просто ревʼювати й заливати.
Для цього PR проревʼюв і залий, будь ласка, burlakvo#1 - я таки виділив час і виправив ворнінґи.
Тоді заллю цей.

@hedrok
Copy link
Member

hedrok commented Oct 20, 2024

Щойно почне заважати, що в тебе немає прав (або я не встигатиму ревʼю, або ти захочеш ревʼювати когось і заливати) - пиши, попросимо тобі права.

Виправлення warning'ів
@hedrok hedrok merged commit 59cb96c into progit:master Oct 20, 2024
1 check passed
@burlakvo
Copy link
Contributor Author

Я не вказав в PR "кодове слово" (resolves #196), аби GitHub автоматично зв'язав його із відповідним завданням. А вручну то зробити я не можу, бо прав немає. Треба, мабуть, їх вручну зв'язати, або, хоча б, закрити завдання

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