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

Remove jekyll #521

Closed
wants to merge 5 commits into from
Closed

Remove jekyll #521

wants to merge 5 commits into from

Conversation

dzencot
Copy link
Collaborator

@dzencot dzencot commented Oct 24, 2022

Тут много всего. Удалён джекил, пофикшены некоторые гайды. Картинки вроде корректно все работают, прошёлся по паре гайдов.

@vercel
Copy link

vercel bot commented Oct 24, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
hexletguides ✅ Ready (Inspect) Visit Preview Oct 24, 2022 at 6:29PM (UTC)

@dzencot
Copy link
Collaborator Author

dzencot commented Oct 24, 2022

Этот пр нужно принимать до других

@trolzen
Copy link
Contributor

trolzen commented Oct 24, 2022

@dzencot Как у вас получилось запросить code review? У меня вроде такая же лычка — Contributor — но в своём PR я не вижу такой возможности.

@dzencot
Copy link
Collaborator Author

dzencot commented Oct 24, 2022

@trolzen нужно сначала назначить ревьюеров, а потом, после создания пр, можно у них запрашивать

@trolzen
Copy link
Contributor

trolzen commented Oct 24, 2022

@dzencot Так вот и вопрос: как их назначить? Если справа вверху в разделе "Reviewers", то у меня там такой возможности нет: ни кнопки, ни карандашика — ничего.

<br><small>Терминал iTerm 2 со split-окнами</small>
<sub><sup>Терминал iTerm 2 со split-окнами</sup></sub>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Чем <small> не угодил? Попробовал на живом сайте так поменять, уж очень мелко получилось. Может, вернуть, как было?

<br><small>Клавиатура терминала ADM-3A</small>
<sub><sup>Клавиатура терминала ADM-3A</sup></sub>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тот же вопрос про <small>.

<br><small>Фрагмент файла конфигурации vim</small>
<sub><sup>Фрагмент файла конфигурации vim</sup></sub>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тот же вопрос про <small>.

image: "/assets/images/git/git-portal.png"
image: assets/images/git/git-portal.png
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Так мы ставим слеш в начале и кавычки вокруг, как в предыдущих файлах, или убираем, как здесь?

Comment on lines -28 to +34
<!--
{/*
Когда браузер встречает тег `<script>` с указанием `src`,
он загружает указанный файл и исполняет его.
Так код получает доступ к содержимому страницы.
-->
*/}

<!-- Все определения внутри становятся глобальны и доступны для всех остальных скриптов -->
{/* Все определения внутри становятся глобальны и доступны для всех остальных скриптов */}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Думаю, здесь всё же надо оставить HTML-комментарии, это же пример кода.

* java: sdkman, jabba
* java: jabba
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Что со sdkman не так?

@@ -58,7 +58,7 @@ make test
```makefile
# Makefile
цель1: # имя цели, поддерживается kebab-case и snake_case
команда1 # для отступа используется табуляция, это важная деталь
команда1 # для отступа используется табуляция, это важная деталь
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

пробел в конце не нужен

Suggested change
команда1 # для отступа используется табуляция, это важная деталь
команда1 # для отступа используется табуляция, это важная деталь

@@ -269,7 +269,7 @@ project-env-generate:

### Дополнительные материалы

* [Руководство по современному Make](https://ru.makefile.site/) — «выжимка» из документации на русском языке;
{/*- [Руководство по современному Make](https://ru.makefile.site/) — «выжимка» из документации на русском языке; -*/}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Почему закомментировано? Если потому, что ru-версия не работает, то, может, дать ссылку на английскую? Или прямую на PDF-ку. Или на веб-архив. Или на github. Или вообще связаться с автором и попросить его восстановить.

@@ -32,7 +32,7 @@ author: Татьяна Федорина

Разберём этапы компиляции на примере вычисления периметра прямоугольника:

```cpp
```c
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это всё же код на C++, под C не скомпилируется.

Suggested change
```c
```cpp

@@ -81,4 +81,4 @@ hexlet.io mail is handled by 5 alt2.aspmx.l.google.com.

Хостинг может быть простым и бесплатным, а может быть платным и требующим специальных знаний и навыков администрирования. Можно начать с размещения на бесплатном хостинге с бесплатным доменом. Но не следует путать регистраторов доменных имён и хостеров. Часто хостеры выступают как посредники и предлагают купить домены через себя, но, в общем случае, регистраторы — это отдельные компании, которые никак не связаны с хостингом.

А получить все необходимые навыки программирования и работы с веб-приложениями можно с нашими наставниками в одной из профессий: https://ru.hexlet.io/professions
А получить все необходимые навыки программирования и работы с веб-приложениями можно с нашими наставниками в одной из профессий: https://ru.hexlet.io/professions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

пробел в конце не нужен

Suggested change
А получить все необходимые навыки программирования и работы с веб-приложениями можно с нашими наставниками в одной из профессий: https://ru.hexlet.io/professions
А получить все необходимые навыки программирования и работы с веб-приложениями можно с нашими наставниками в одной из профессий: https://ru.hexlet.io/professions

@@ -6,7 +6,7 @@ image: "/assets/images/logging/cover.png"
author: Кирилл Мокевнин
---

Известно, что программисты проводят много времени, отлаживая свои программы, пытаясь разобраться, почему они не работают — или работают неправильно. Когда говорят про отладку, обычно подразумевают либо отладочную печать, либо использование специальных программ – дебагеров. С их помощью отслеживается выполнение кода по шагам, во время которого видно, как меняется содержимое переменных. Эти способы хорошо работают в небольших программах, но в реальных приложениях быстро становятся неэффективными.
Известно, что программисты проводят много времени, отлаживая свои программы, пытаясь разобраться, почему они не работают — или работают неправильно. Когда говорят про отладку, обычно подразумевают либо отладочную печать, либо использование специальных программ – дебагеров. С их помощью отслеживается выполнение кода по шагам, во время которого видно, как меняется содержимое переменных. Эти способы хорошо работают в небольших программах, но в реальных приложениях быстро становятся неэффективными.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

пробел в конце не нужен

Suggested change
Известно, что программисты проводят много времени, отлаживая свои программы, пытаясь разобраться, почему они не работают — или работают неправильно. Когда говорят про отладку, обычно подразумевают либо отладочную печать, либо использование специальных программ – дебагеров. С их помощью отслеживается выполнение кода по шагам, во время которого видно, как меняется содержимое переменных. Эти способы хорошо работают в небольших программах, но в реальных приложениях быстро становятся неэффективными.
Известно, что программисты проводят много времени, отлаживая свои программы, пытаясь разобраться, почему они не работают — или работают неправильно. Когда говорят про отладку, обычно подразумевают либо отладочную печать, либо использование специальных программ – дебагеров. С их помощью отслеживается выполнение кода по шагам, во время которого видно, как меняется содержимое переменных. Эти способы хорошо работают в небольших программах, но в реальных приложениях быстро становятся неэффективными.

@@ -30,7 +30,7 @@ author: Кирилл Мокевнин
И для всего этого многообразия систем существует единое решение — логирование. В простейшем случае логирование сводится к файлу на диске, куда разные программы записывают (логируют) свои действия во время работы. Такой файл называют логом или журналом. Как правило, внутри лога одна строчка соответствует одному действию.

```shell
# Формат: ip-address / date / HTTP-method / uri / response code / body size
# Формат: ip-address / date / HTTP-method / uri / response code / body size
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

опять лишний пробел

Suggested change
# Формат: ip-address / date / HTTP-method / uri / response code / body size
# Формат: ip-address / date / HTTP-method / uri / response code / body size

@@ -132,7 +132,7 @@ DEBUG=http server.js

## Ротация логов

Со временем количество логов становится большим, и с ними нужно что-то делать. Для этого используется ротация логов. Иногда за это отвечает сама программа, но чаще — внешнее приложение, задачей которого является чистка. Эта программа по необходимости разбивает логи на более мелкие файлы, сжимает, перемещает и, если нужно, удаляет. Подобная система встроена в любую операционную систему для работы с логами самой системы и внешних программ, которые могут встраиваться в нее.
Со временем количество логов становится большим, и с ними нужно что-то делать. Для этого используется ротация логов. Иногда за это отвечает сама программа, но чаще — внешнее приложение, задачей которого является чистка. Эта программа по необходимости разбивает логи на более мелкие файлы, сжимает, перемещает и, если нужно, удаляет. Подобная система встроена в любую операционную систему для работы с логами самой системы и внешних программ, которые могут встраиваться в нее.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ещё один

Suggested change
Со временем количество логов становится большим, и с ними нужно что-то делать. Для этого используется ротация логов. Иногда за это отвечает сама программа, но чаще — внешнее приложение, задачей которого является чистка. Эта программа по необходимости разбивает логи на более мелкие файлы, сжимает, перемещает и, если нужно, удаляет. Подобная система встроена в любую операционную систему для работы с логами самой системы и внешних программ, которые могут встраиваться в нее.
Со временем количество логов становится большим, и с ними нужно что-то делать. Для этого используется ротация логов. Иногда за это отвечает сама программа, но чаще — внешнее приложение, задачей которого является чистка. Эта программа по необходимости разбивает логи на более мелкие файлы, сжимает, перемещает и, если нужно, удаляет. Подобная система встроена в любую операционную систему для работы с логами самой системы и внешних программ, которые могут встраиваться в нее.

Comment on lines -51 to +56
<!---
{/*
## Related guides

1. [What is Vagrant](/vagrant/). Vagrant allows creating and configuring lightweight, repeatable, and portable development environments for virtual machines.
2. [How to work with Linux using Windows](/ubuntu-linux-in-windows/). Instructions for installing Ubuntu Linux on Windows using various virtualization technologies.
--->
*/}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Может, здесь вообще комментирование убрать? Статьи-то такие есть на сайте. Или дело в том, что у них нет английской версии?

@@ -56,7 +56,7 @@ Later, we'll see that different tools use different approaches: some use deltas

Another good example of how to use version control is text editors, particularly online ones.

<!-- ![google docs, version history](/assets/images/git/2.png) -->
{/* ![google docs, version history](/assets/images/git/2.png) */}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут я тоже не понимаю, зачем картинка закомментирована, т.к. в русской версии она есть. К тому же, без нёё непонятно, о чём речь в предложениях выше и ниже. Предлагаю раскомментировать.

Suggested change
{/* ![google docs, version history](/assets/images/git/2.png) */}
![google docs, version history](/assets/images/git/2.png)

@@ -133,7 +133,7 @@ RCS, SCCS
* Work with each file separately
* Local work only

<!-- ![version control systems, first generation](/assets/images/git/4.png) -->
{/* ![version control systems, first generation](/assets/images/git/4.png) */}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

то же здесь

Suggested change
{/* ![version control systems, first generation](/assets/images/git/4.png) */}
![version control systems, first generation](/assets/images/git/4.png)

@@ -156,7 +156,7 @@ Git, Bazaar, Mercurial

A server is only used to store the reference repository. In fact, all repository copies are equal and can exchange data in any direction.

<!-- ![version control systems, third generation](/assets/images/git/6.png) -->
{/* ![version control systems, third generation](/assets/images/git/6.png) */}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

и здесь

Suggested change
{/* ![version control systems, third generation](/assets/images/git/6.png) */}
![version control systems, third generation](/assets/images/git/6.png)

Comment on lines +88 to 90
````
А иногда нужно вставить целый блок кода:

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Если переделывать, то тогда текст нужно вынести. И тут и ниже точно должны быть четверные, не тройные тики? Они точно сработают? Может, надо заменить на тильды?

Suggested change
````
А иногда нужно вставить целый блок кода:
А иногда нужно вставить целый блок кода:
````

Comment on lines +32 to +33
2. Повторяемость. Пересоздать рабочую среду можно за считанные минуты набрав буквально одну команду. Любое изменение распространяется сразу для всех.
3. Переносимость. Окружение разворачивается под любой системой одним универсальным способом.
Copy link
Contributor

@trolzen trolzen Oct 24, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вот это лишнее, это же Markdown.

Suggested change
2. Повторяемость. Пересоздать рабочую среду можно за считанные минуты набрав буквально одну команду. Любое изменение распространяется сразу для всех.
3. Переносимость. Окружение разворачивается под любой системой одним универсальным способом.
1. Повторяемость. Пересоздать рабочую среду можно за считанные минуты набрав буквально одну команду. Любое изменение распространяется сразу для всех.
1. Переносимость. Окружение разворачивается под любой системой одним универсальным способом.

@trolzen
Copy link
Contributor

trolzen commented Oct 24, 2022

Я хоть и не ревьюер этого проекта, но просмотрел изменения и набросал комментариев. Все они относятся к изменениям в контенте. Правда, что-то они как-то некрасиво тут на вкладке "Conversation" легли, длинная простыня получилась.

Кстати, мне кажется, не стоило включать правки контента в такой масштабный PR, круто меняющий структуру проекта, и так там очень много изменений, которые сложно отслеживать.

@mokevnin
Copy link
Member

Тут много всего. Удалён джекил, пофикшены некоторые гайды. Картинки вроде корректно все работают, прошёлся по паре гайдов.

Давай таки разделим. Удаление отдельно, гайды отдельно

@dzencot
Copy link
Collaborator Author

dzencot commented Oct 25, 2022

@trolzen ага, спасибо за ревью! Вообще я гайды просто скопировал и поправил только то, что не собиралось. В общем разделю пр. Правки по гайдом будут отдельно

@dzencot
Copy link
Collaborator Author

dzencot commented Oct 25, 2022

Этот пр закрываю. Вместо него будут отдельные пры:
#522

@dzencot dzencot closed this Oct 25, 2022
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.

None yet

3 participants