Update README.md #120
Update README.md #120
Conversation
WalkthroughИзменения в документе Changes
Assessment against linked issues
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 9
Out of diff range and nitpick comments (3)
README.md (3)
Line range hint
212-213: Списки должны быть окружены пустыми строками.213a214 +
Line range hint
213-215: Используйте символ*для маркированного списка и исправьте отступы.- * Если ваши тесты размещены в отдельных расширениях, скопируйте файл `./resources/yaxunit.json` из текущей библиотеки в свой репозиторий (`./tools/yaxunit.json`) и перечислите в нем имена ваших расширений. - * Если используется собственный файл `tools/yaxunit.json`, то значение параметра reportPath в нем должно быть равно `./build/out/yaxunit/junit.xml` + * Если ваши тесты размещены в отдельных расширениях, скопируйте файл `./resources/yaxunit.json` из текущей библиотеки в свой репозиторий (`./tools/yaxunit.json`) и перечислите в нем имена ваших расширений. + * Если используется собственный файл `tools/yaxunit.json`, то значение параметра reportPath в нем должно быть равно `./build/out/yaxunit/junit.xml`
Line range hint
29-30: Используйте форматированный URL.- См. [памятку по слоям и последовательности сборки](https://github.com/firstBitMarksistskaya/onec-docker/blob/feature/first-bit/Layers.md). + См. [памятку по слоям и последовательности сборки](https://github.com/firstBitMarksistskaya/onec-docker/blob/feature/first-bit/Layers.md).
|
А зачем ты нумерацию поменял? |
|
@nixel2007 нумерация сбивалась, но сейчас я внес исправление, благодаря которому она заработала как надо |
README.md
Outdated
| * В случае загрузки из исходников - необходимо указать путь к исходникам расширения | ||
| * В случае загрузки cfe - Укажите путь по которому будет скачан cfe. На данный момент можно указывать как локальный путь, так и url для скачивания cfe(Прим.: https://github.com/bia-technologies/yaxunit/releases/download/23.05/YAXUNIT-23.05.cfe) |
There was a problem hiding this comment.
Добавьте пустую строку перед списком для улучшения читаемости.
187a188
+Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.
| * В случае загрузки из исходников - необходимо указать путь к исходникам расширения | |
| * В случае загрузки cfe - Укажите путь по которому будет скачан cfe. На данный момент можно указывать как локальный путь, так и url для скачивания cfe(Прим.: https://github.com/bia-technologies/yaxunit/releases/download/23.05/YAXUNIT-23.05.cfe) | |
| * В случае загрузки из исходников - необходимо указать путь к исходникам расширения | |
| * В случае загрузки cfe - Укажите путь по которому будет скачан cfe. На данный момент можно указывать как локальный путь, так и url для скачивания cfe(Прим.: https://github.com/bia-technologies/yaxunit/releases/download/23.05/YAXUNIT-23.05.cfe) |
| - В случае загрузки cfe - Укажите путь по которому будет скачан cfe. На данный момент можно указывать как локальный путь, так и url для скачивания cfe(Прим.: https://github.com/bia-technologies/yaxunit/releases/download/23.05/YAXUNIT-23.05.cfe) | ||
| * В случае загрузки из исходников - необходимо указать путь к исходникам расширения | ||
| * В случае загрузки cfe - Укажите путь по которому будет скачан cfe. На данный момент можно указывать как локальный путь, так и url для скачивания cfe(Прим.: https://github.com/bia-technologies/yaxunit/releases/download/23.05/YAXUNIT-23.05.cfe) | ||
| 1. Укажите этапы сборки, на которых должно быть загружено расширение (`initInfobase` -> `extensions` -> `stages`). Если оставить это поле пустым, то расширение будет загружено на этапе `initInfobase` и будет активно на всех последующих этапах. В противном случае расширение будет использоваться только на перечисленных этапах. | ||
|
|
||
| Пример конфигурации для загрузки расширений: | ||
| ```json |
There was a problem hiding this comment.
Добавьте пустые строки вокруг блока кода для улучшения читаемости.
192a193
+
194a195
+Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.
| ```json | |
| ```json |
</details>
<!-- suggestion_end -->
<!-- This is an auto-generated comment by CodeRabbit -->
There was a problem hiding this comment.
Actionable comments posted: 0
Out of diff range and nitpick comments (3)
README.md (3)
Line range hint
30-30: Используйте Markdown для URL, чтобы избежать использования голых ссылок.- https://github.com/Stepa86/stebi + [stebi](https://github.com/Stepa86/stebi) - https://github.com/firstBitMarksistskaya/onec-docker/blob/feature/first-bit/Layers.md + [памятку по слоям и последовательности сборки](https://github.com/firstBitMarksistskaya/onec-docker/blob/feature/first-bit/Layers.md) - https://jenkins.io/doc/book/pipeline/shared-libraries/#using-libraries + [Инструкция по подключению библиотеки](https://jenkins.io/doc/book/pipeline/shared-libraries/#using-libraries) - https://github.com/bia-technologies/yaxunit/releases/download/23.05/YAXUNIT-23.05.cfe + [YAXUNIT-23.05.cfe](https://github.com/bia-technologies/yaxunit/releases/download/23.05/YAXUNIT-23.05.cfe)Also applies to: 49-49, 116-116, 189-189
Line range hint
73-73: Удалите лишние пустые строки для улучшения читаемости.72a73 -
Line range hint
9-9: Удалите лишние пробелы в конце строк.- Строка с лишними пробелами + Строка без лишних пробеловAlso applies to: 85-85, 112-112, 125-125, 162-162
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- README.md (1 hunks)
Additional comments not posted (6)
README.md (6)
177-177: Добавьте пустую строку перед списком для улучшения читаемости.
179-179: Используйте символ*для маркированного списка.
185-186: Используйте символ*для маркированного списка и исправьте отступы.
188-189: Используйте символ*для маркированного списка и исправьте отступы.
189-190: Списки должны быть окружены пустыми строками.
200-201: Используйте символ*для маркированного списка.
|
Спасибо! |
fixes #119