feat: new scenario ПроверкаНаличияАртефактовGit#54
feat: new scenario ПроверкаНаличияАртефактовGit#54Kirill wants to merge 6 commits intobia-technologies:developfrom
Conversation
ORAIS-1949: Новый сценарий сортировка состава подписок на события. Closes ORAIS-1949 See merge request orais/precommit1C-fork-bia!77
Релиз 25.1
Проверка на наличие артефактов после разрешения конфликтов
|
Warning Rate limit exceeded@Kirill has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 11 minutes and 38 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
| ПаттерныКонфликтов = Новый Массив(); | ||
| ПаттерныКонфликтов.Добавить("^<<<<<<<\s"); | ||
| ПаттерныКонфликтов.Добавить("^=======\s"); | ||
| ПаттерныКонфликтов.Добавить("^=======\n$"); | ||
| ПаттерныКонфликтов.Добавить("^=======\r\n$"); | ||
| ПаттерныКонфликтов.Добавить("^>>>>>>>\s"); |
There was a problem hiding this comment.
Почему массивом, а не одной регуряркой (через |)?
There was a problem hiding this comment.
Ну и тесты хотелось бы увидеть, в том числе такие, в которых есть подобные строки в строковых литералах
|
@Kirill |
4 participants
Проверка на наличие артефактов после разрешения конфликтов