Skip to content

Update bsl-parser dependency to version 0.29.0#3651

Merged
nixel2007 merged 1 commit intodevelopfrom
nixel2007-patch-2
Dec 9, 2025
Merged

Update bsl-parser dependency to version 0.29.0#3651
nixel2007 merged 1 commit intodevelopfrom
nixel2007-patch-2

Conversation

@nixel2007
Copy link
Member

@nixel2007 nixel2007 commented Dec 9, 2025

Описание

Связанные задачи

Closes

Чеклист

Общие

  • Ветка PR обновлена из develop
  • Отладочные, закомментированные и прочие, не имеющие смысла участки кода удалены
  • Изменения покрыты тестами
  • Обязательные действия перед коммитом выполнены (запускал команду gradlew precommit)

Для диагностик

  • Описание диагностики заполнено для обоих языков (присутствуют файлы для обоих языков, для русского заполнено все подробно, перевод на английский можно опустить)

Дополнительно

Summary by CodeRabbit

  • Chores
    • Updated BSL parser dependency to version 0.29.0.

✏️ Tip: You can customize this high-level summary in your review settings.

Copilot AI review requested due to automatic review settings December 9, 2025 18:31
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 9, 2025

Walkthrough

The BSL parser dependency is updated from version 0.28.0 to 0.29.0 in build.gradle.kts. The exclusions for com.ibm.icu, ST4, and antlr-runtime remain unchanged. No other modifications to build logic or configuration are present.

Changes

Cohort / File(s) Summary
Dependency Version Update
build.gradle.kts
BSL parser dependency version bumped from 0.28.0 to 0.29.0

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A version bump, so small and neat,
From twenty-eight to twenty-nine we leap!
The parser grows with fresh delight,
Our build tools dance, dependencies tight! 📦✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title directly and clearly describes the main change: updating the bsl-parser dependency from version 0.28.0 to 0.29.0, which is reflected in the build.gradle.kts file modification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch nixel2007-patch-2

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89370e0 and 1d5fdd1.

📒 Files selected for processing (1)
  • build.gradle.kts (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
build.gradle.kts

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

build.gradle.kts: Use Gradle with Kotlin DSL for build configuration
Be cautious with external dependencies and review security alerts from CodeQL and Dependabot

Files:

  • build.gradle.kts
🧠 Learnings (2)
📓 Common learnings
Learnt from: CR
Repo: 1c-syntax/bsl-language-server PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-27T07:17:33.726Z
Learning: Applies to build.gradle.kts : Be cautious with external dependencies and review security alerts from CodeQL and Dependabot
Learnt from: theshadowco
Repo: 1c-syntax/bsl-language-server PR: 3610
File: src/main/java/com/github/_1c_syntax/bsl/languageserver/folding/QueryPackageFoldingRangeSupplier.java:26-26
Timestamp: 2025-11-19T09:02:03.154Z
Learning: В проекте bsl-language-server используется кастомная версия ANTLR4 от 1c-syntax (io.github.1c-syntax:antlr4), которая включает базовый класс org.antlr.v4.runtime.Tokenizer с методами getAst() и getTokens(). SDBLTokenizer и BSLTokenizer наследуются от этого базового класса.
📚 Learning: 2025-11-27T07:17:33.726Z
Learnt from: CR
Repo: 1c-syntax/bsl-language-server PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-27T07:17:33.726Z
Learning: Applies to build.gradle.kts : Be cautious with external dependencies and review security alerts from CodeQL and Dependabot

Applied to files:

  • build.gradle.kts
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (13)
  • GitHub Check: Analyse
  • GitHub Check: build (25, macOS-latest)
  • GitHub Check: build (21, macOS-latest)
  • GitHub Check: build (25, ubuntu-latest)
  • GitHub Check: build (17, macOS-latest)
  • GitHub Check: build (25, windows-latest)
  • GitHub Check: build (21, ubuntu-latest)
  • GitHub Check: build (17, windows-latest)
  • GitHub Check: build (17, ubuntu-latest)
  • GitHub Check: build (21, windows-latest)
  • GitHub Check: build
  • GitHub Check: Benchmark
  • GitHub Check: Sentry
🔇 Additional comments (1)
build.gradle.kts (1)

85-85: Verify v0.29.0 compatibility and ensure comprehensive testing.

This version bump requires explicit verification of compatibility, particularly given recent ANTLR-related changes referenced in the project's PR history. Per coding guidelines, be cautious with external dependencies and review security alerts.

Please confirm the following before merging:

  1. Release notes & breaking changes: Review the bsl-parser v0.29.0 release notes to identify any breaking changes or grammar modifications that may require code adjustments in bsl-language-server.
  2. Compatibility with recent work: Ensure this update is compatible with PR Новые Антлр и парсер. Отказ от BSLParserRuleContext #3610 ("New ANTLR and parser, rejection of BSLParserRuleContext").
  3. Build & test verification: Confirm that the full build and test suite passes locally with this change.
  4. Security advisory check: Run CodeQL or Dependabot to verify no new vulnerabilities are introduced.

The PR description and checklist suggest verification steps have not yet been completed (all items unchecked). Please document test results in the 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 and usage tips.

@nixel2007 nixel2007 enabled auto-merge December 9, 2025 18:32
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 9, 2025

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