Semver #235
Semver #235
Conversation
Обновлена версия OPM до 1.4.0 и обновлены зависимости - Обновлена версия OPM до 1.4.0 в packagedef и КонстантыOpm.os - Обновлены версии зависимостей в packagedef: - asserts: 1.3.0 -> 1.4.0 - logos: 1.3.0 -> 1.7.1 - cli: 0.10.2 -> 0.11.0 - tempfiles: 1.0.0 -> 1.1.1 - gitrunner: 1.6.2 -> 1.7.1 - Добавлена зависимость "semver" версии 1.0.0 в packagedef - Добавлено использование "semver" в src\core\Модули\РаботаСОписаниемПакета.os и src\core\Классы\КэшУстановленныхПакетов.os - Исправлена ошибка в КэшУстановленныхПакетов.os, где УстановленнаяВерсия приравнивалась к 0, теперь она приравнивается к "0". - Внесены изменения в логику сравнения версий в src\core\Модули\РаботаСОписаниемПакета.os и src\core\Классы\КэшУстановленныхПакетов.os для использования semver
Сравнение версии движка сделано в начале по старому алгортиму, и если не получится то по семверу.
|
|
||
| Попытка | ||
| // До приведения версии движка к семверу | ||
| РезультатСравнения = РаботаСВерсиями.СравнитьВерсии(ТребуемаяВерсияСреды, ВерсияСреды); |
There was a problem hiding this comment.
Ночник приведен к семверу начиная с 2.0.0-rc.7+591, однако старые версии 1.x остаются четырехзначными
There was a problem hiding this comment.
Всё так, именно по этому тут старый вариант в попытке, а в исключении semver
There was a problem hiding this comment.
Может просто тогда нафиг не нужен семвер, я про это?
There was a problem hiding this comment.
Как вариант, семвер делать в попытке, а в исключении старый вариант. И пометить его как устаревший
There was a problem hiding this comment.
Старый парсер на 4 компонента раcсчитан и падает на "2.0.0.rc", а semver не падает а корректно разбирает и сравнивает, кроме того с семвером я смогу в ванюните указать с какого RC он будет работать и это будет корректно проверятся например.
ИМХО нужен
There was a problem hiding this comment.
Ну и с семвером можно ещё например вот такие штуки делать:
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
WalkthroughВнесены изменения в зависимости и версионирование пакета, обновлены значения версий в константах и fallback-значениях, а также добавлена и интегрирована библиотека Changes
Sequence Diagram(s)sequenceDiagram
participant Пользователь
participant КэшУстановленныхПакетов
participant Версии (semver)
Пользователь->>КэшУстановленныхПакетов: Проверить, установлен ли пакет с минимальной версией
КэшУстановленныхПакетов->>Версии (semver): Сравнить версии (установленная, минимальная)
Версии (semver)-->>КэшУстановленныхПакетов: Результат сравнения
КэшУстановленныхПакетов-->>Пользователь: true/false
sequenceDiagram
participant РаботаСОписаниемПакета
participant Версии (semver)
participant РаботаСВерсиями (старый модуль)
РаботаСОписаниемПакета->>РаботаСВерсиями: Сравнить версии (try)
alt Ошибка
РаботаСОписаниемПакета->>Версии (semver): Сравнить версии (fallback)
end
РаботаСОписаниемПакета-->>РаботаСОписаниемПакета: Использовать результат для проверки
Poem
Note ⚡️ AI Code Reviews for VS Code, Cursor, WindsurfCodeRabbit now has a plugin for VS Code, Cursor and Windsurf. This brings AI code reviews directly in the code editor. Each commit is reviewed immediately, finding bugs before the PR is raised. Seamless context handoff to your AI code agent ensures that you can easily incorporate review feedback. 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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. 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 using PR comments)
Other keywords and placeholders
CodeRabbit Configuration 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: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
.github/workflows/main.yml(1 hunks)packagedef(1 hunks)src/core/Классы/КэшУстановленныхПакетов.os(3 hunks)src/core/Модули/КонстантыOpm.os(1 hunks)src/core/Модули/РаботаСОписаниемПакета.os(3 hunks)
🔇 Additional comments (11)
src/core/Модули/КонстантыOpm.os (1)
27-27: Обновление версии выглядит корректно.Изменение версии продукта с "1.3.0" на "1.4.0" соответствует общему обновлению версии в рамках данного PR.
.github/workflows/main.yml (1)
11-11: Хорошее расширение тестовой матрицы.Добавление версии 'dev' в матрицу тестирования повышает покрытие тестов и помогает выявить потенциальные проблемы совместимости с развиваемыми версиями oscript.
src/core/Классы/КэшУстановленныхПакетов.os (2)
4-4: Добавление библиотеки semver для улучшенной работы с версиями.Импорт библиотеки semver соответствует общей цели PR по улучшению обработки версий в проекте.
33-33: Исправление типа данных для версии по умолчанию.Изменение с числового
0на строковое"0"обеспечивает согласованность типов данных при работе с версиями, что важно для корректного сравнения версий.packagedef (4)
16-16: Синхронизация версии fallback с константами.Обновление резервной версии с "1.3.0" на "1.4.0" корректно синхронизировано с изменением в файле констант.
30-30: Добавление библиотеки semver.Добавление зависимости semver 1.0.0 поддерживает улучшения в обработке версий, видимые в других файлах проекта.
31-33: Реорганизация зависимостей для разработки.Хорошая реорганизация зависимостей:
- asserts перемещен в зависимости разработки (с обновлением до 1.4.0)
- Добавлены 1bdd и 1testrunner для улучшения возможностей тестирования
Это логично, поскольку эти библиотеки используются только во время разработки и тестирования.
25-28:❓ Verification inconclusive
Обновление версий зависимостей.
Обновления версий зависимостей выглядят разумно:
- logos: значительное обновление с 1.3.0 до 1.7.1
- cli: минорное обновление с 0.10.2 до 0.11.0
- tempfiles: патч обновление с 1.0.0 до 1.1.1
- gitrunner: минорное обновление с 1.6.2 до 1.7.1
Рекомендуется проверить совместимость обновлений, особенно значительного обновления logos.
🌐 Web query:
Проверить документацию по совместимости версий logos library для OneScript от версии 1.3.0 до 1.7.1💡 Result:
Библиотека logos предназначена для логирования в стиле log4j и активно используется в сообществе OneScript. Документация по совместимости версий библиотеки logos с версиями OneScript от 1.3.0 до 1.7.1 не представлена в открытых источниках.
Однако, согласно информации из репозитория библиотеки logos на GitHub, она совместима с OneScript версии 1.3.0 и выше. В частности, в разделе "Потеря совместимости" указано, что внешние компоненты от версии 1.x не будут работать без перекомпиляции или адаптации кода. (github.com)
Таким образом, для использования библиотеки logos с версиями OneScript от 1.3.0 до 1.7.1 рекомендуется проверить совместимость конкретной версии библиотеки с вашей версией OneScript. Если возникают проблемы, возможно, потребуется перекомпиляция или адаптация кода библиотеки.
Citations:
Проверьте и протестируйте функциональность логирования после обновления logos до 1.7.1
- Файл:
packagedef, строки 25–28 (обновлены зависимости):.ЗависитОт("logos", "1.7.1") .ЗависитОт("cli", "0.11.0") .ЗависитОт("tempfiles", "1.1.1") .ЗависитОт("gitrunner", "1.7.1")- Согласно репозиторию GitHub библиотеки logos, версии 1.x совместимы с OneScript 1.3.0 и выше, но могут требовать перекомпиляции или адаптации кода.
- Обновления
cli,tempfilesиgitrunnerявляются минорными или патчевыми и, как правило, не должны нарушить совместимость.Рекомендуется:
- После обновления запустить сценарии, использующие логирование, и убедиться в корректной работе.
- При обнаружении проблем – перекомпилировать или адаптировать библиотеку logos.
src/core/Модули/РаботаСОписаниемПакета.os (3)
2-2: Добавление зависимости semver выглядит корректно.Подключение библиотеки semver для работы с семантическим версионированием является логичным улучшением.
66-71: Fallback механизм реализован корректно.Использование try-catch блока для обеспечения обратной совместимости со старыми четырехзначными версиями является правильным решением, учитывая обсуждение в предыдущих комментариях.
73-73: Логика сравнения версий корректна.Использование переменной
РезультатСравнениявместо прямого вызова функции в условии улучшает читаемость кода.
Обновлена версия OPM до 1.4.0 и обновлены зависимости
Summary by CodeRabbit