Skip to content

Исправил разбор вывода с несколькими пустыми строками - #74

Merged
arkuznetsov merged 1 commit into
arkuznetsov:masterfrom
GenAcid:master
Oct 18, 2026
Merged

arkuznetsov merged 1 commit into
arkuznetsov:masterfrom
GenAcid:master

Conversation

@GenAcid

@GenAcid GenAcid commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Платформа 8.3.27.2214 в конце вывода rac два перевода строки. Из-за этого в МассивРезультатов передается лишнее пустое соотвествие.

Пример вывода
`C:\Program Files\1cv8\8.3.27.2214\bin>rac.exe servername.host infobase --cluster=18250a24-74e4-4e62-ae35-718b86ae24c4 summary list
infobase : 55108dff-bb8e-4352-8504-c227184a71c8
name : Base1
descr :

infobase : 235fbfc6-2ab4-4998-8dfd-8fe45e4ca87b
name : Base2
descr :

C:\Program Files\1cv8\8.3.27.2214\bin>`

Summary by CodeRabbit

  • Исправления
    • При разборе вывода команды пустые строки больше не создают пустые описания результатов.

Если в выводе было несколько пустых строк, к результатам добавлялось пустое соответствие
@coderabbitai

coderabbitai Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: c650fc5c-4708-4222-aba9-5f6c0ff1a6ca

📥 Commits

Reviewing files that changed from the base of the PR and between d2a8d0c and c739eff.

📒 Files selected for processing (1)
  • src/Модули/Служебный.os

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

В функции РазобратьВыводКоманды добавлена проверка заполненности описания. При пустой строке описание добавляется в массив результатов и сбрасывается только при наличии значения.

Changes

Разбор вывода команды

Layer / File(s) Summary
Проверка описания перед добавлением
src/Модули/Служебный.os
При встрече пустой строки описание добавляется в МассивРезультатов и сбрасывается только при условии ЗначениеЗаполнено(Описание). Ранее добавление и сброс происходили безусловно.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to c739e

The change correctly avoids adding empty results for repeated trailing blank lines. No concrete merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Заголовок точно описывает основное изменение: исправление разбора вывода с несколькими пустыми строками и предотвращение добавления пустого результата.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Я, кролик, прыгаю по строчкам пустым,
Проверку встречаю — и радость в ушах.
Заполнено? Блок отправляется в список,
Пустое не копится больше в руках.
Морковку грызу — разбор стал аккуратным!

Comment @coderabbitai help to get the list of available commands.

@arkuznetsov
arkuznetsov merged commit 706bc6c into arkuznetsov:master Oct 18, 2026
1 check passed
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