Skip to content

Вернуть раздел Index - #367

Merged
Morganov merged 6 commits into
progit:masterfrom
srgizh:fix-index
May 11, 2021
Merged

Вернуть раздел Index#367
Morganov merged 6 commits into
progit:masterfrom
srgizh:fix-index

Conversation

@srgizh

@srgizh srgizh commented Feb 3, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor

#363 можно удалить

@srgizh
srgizh force-pushed the fix-index branch 2 times, most recently from a117321 to 6b7f204 Compare February 3, 2021 01:40
@srgizh

srgizh commented Feb 3, 2021

Copy link
Copy Markdown
Contributor Author

Результат:

image

@srgizh srgizh mentioned this pull request Feb 3, 2021
Closed
@srgizh
srgizh force-pushed the fix-index branch 4 times, most recently from c087466 to 2cda839 Compare February 8, 2021 21:31
@srgizh
srgizh force-pushed the fix-index branch 3 times, most recently from 50ef687 to ebe707a Compare February 14, 2021 06:50
@srgizh

srgizh commented Feb 14, 2021

Copy link
Copy Markdown
Contributor Author

Разрешил конфликты

@Morganov

Morganov commented Feb 28, 2021 •
edited
Loading

Copy link
Copy Markdown
Member

Книга не собирается локально: pdf файл не генерируется. Возможно, перебазирование решит проблему.
Также, возможно, лишний файл book/index.asc.

@srgizh

srgizh commented Mar 1, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Книга не собирается локально: pdf файл не генерируется. Возможно, перебазирование решит проблему.

Никогда не видел проблем именно со сборкой. Периодически валится проверка внешних ссылок, но это не моя вина.
Повторная сборка через некоторое время обычно проходит хорошо.

Также, возможно, лишний файл book/index.asc.

Да. Это похоже на лишний. Он нигде не используется.

@srgizh

srgizh commented Mar 1, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor Author
  • Перебазировал на актуальный master
  • Удалил неиспользуемый файл book/index.asc
  • Проверил локальную сборку, ошибок не возникло
  • Отправил запрос на изменения в английской версии для решения проблемы с заголовком несуществующего индекса

@Morganov

Morganov commented Mar 7, 2021

Copy link
Copy Markdown
Member 
* Отправил запрос на изменения в английской версии для решения проблемы с заголовком несуществующего индекса

Добавьте ссылку, а то непонятно о чём речь.

@Morganov

Morganov commented Mar 7, 2021

Copy link
Copy Markdown
Member

Книга не собирается локально: pdf файл не генерируется. Возможно, перебазирование решит проблему.

Никогда не видел проблем именно со сборкой. Периодически валится проверка внешних ссылок, но это не моя вина.
Повторная сборка через некоторое время обычно проходит хорошо.

Проблема была не в PR. Сборка PDF требует много ресурсов и процесс может быть принудительно остановлен системой при достижинии лимита.

@srgizh

srgizh commented Mar 8, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor Author 
* Отправил запрос на слияние в английской версии для решения проблемы с заголовком несуществующего индекса

Добавьте ссылку, а то непонятно о чём речь.

progit/progit2#1621
Это копия коммита 173ec4f Индекс только в pdf

@srgizh
srgizh force-pushed the fix-index branch 8 times, most recently from 790515e to 2888232 Compare March 11, 2021 03:25
@srgizh
srgizh force-pushed the fix-index branch 2 times, most recently from 1ac24d9 to b52e52d Compare March 12, 2021 08:15
@srgizh srgizh changed the title Перевод раздела Index Вернуть раздел Index Apr 7, 2021
@Morganov Morganov added ready Ready to merge and removed in-verification labels May 11, 2021
@Morganov
Morganov merged commit 4c45d90 into progit:master May 11, 2021
@srgizh
srgizh deleted the fix-index branch May 15, 2021 03:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Morganov Morganov Morganov approved these changes

Assignees

No one assigned

Labels

ready Ready to merge

Projects

None yet

Milestone

No milestone

Development

See More

Successfully merging this pull request may close these issues.

2 participants

@srgizh @Morganov