Skip to content

build(deps): bump golang from 1.26-bookworm to 1.27-bookworm in /adapters/telegram - #70

Open
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/docker/adapters/telegram/golang-1.27-bookworm
Open

build(deps): bump golang from 1.26-bookworm to 1.27-bookworm in /adapters/telegram#70
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/docker/adapters/telegram/golang-1.27-bookworm

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Aug 24, 2026

Copy link
Copy Markdown
Contributor

Bumps golang from 1.26-bookworm to 1.27-bookworm.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps golang from 1.26-bookworm to 1.27-bookworm.

---
updated-dependencies:
- dependency-name: golang
  dependency-version: 1.27-bookworm
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file docker Docker image dependencies labels Aug 24, 2026

@DuckBugAgent DuckBugAgent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Раунд 1 из 3 · полное ревью · консистентность Go-тулчейна и связь с упавшим CI

Ревью

Замечаний по изменениям не нашёл.

  • Коротко: Диff ровно соответствует заголовку — одна строка build-стадии adapters/telegram/Dockerfile:11, и оба multi-arch билда образов в CI зелёные, так что тег golang:1.27-bookworm существует и модуль под ним собирается.
  • Риски/вопросы:
    • Проверил go.mod: директива go 1.26.5 не требует правки — тулчейн 1.27 собирает модуль с более старой директивой, отдельного toolchain-пина в go.mod нет. Runtime-стадия (node:24-bookworm-slim) Go вообще не содержит, поэтому бамп затрагивает только компиляцию бинаря.
    • Расхождение тулчейнов после мержа: adapters/vk/Dockerfile:11 остаётся на golang:1.26-bookworm, а tools/Dockerfile:1 (dev-tools образ, в котором CI гоняет lint/tests/build/govulncheck) — на golang:1.26.5. То есть релизный бинарь telegram будет собран компилятором, который ни один гейт в CI не прогоняет. Для vk PR от Dependabot придёт отдельно (директория есть в .github/dependabot.yml), а вот /tools в конфиге Dependabot не описана вовсе — её придётся поднимать вручную отдельным PR.
    • Проверка «Security & Dependency Scanning» на head SHA красная, логов у меня нет. По коду связь с диффом не устанавливается: джоба гоняет govulncheck внутри dev-tools образа (golang:1.26.5) и Trivy fs-скан по репозиторию — ни то, ни другое не читает build-стадию adapters/telegram/Dockerfile. Причину не приписываю; стоит сверить, красная ли та же проверка на main (govulncheck ставится как @latest, поэтому новая advisory или обновление БД Trivy красят её без изменений в коде).
    • Тег 1.27-bookworm плавающий по патчам — как и прежний 1.26-bookworm, так что воспроизводимость сборки не меняется, но и не фиксируется по digest.

Арбитр ревью: остановлено — блокирующего не осталось, PR готов к мерджу человеком.

Причина: Раунд 1 из 3: ревьювер вернул вердикт ok без единой находки, диф — одна строка бампа базового образа в adapters/telegram/Dockerfile, и оба multi-arch билда в CI зелёные, то есть тег существует и модуль под ним собирается. Блокирующих находок (BLOCKER/MAJOR/MINOR) нет — держать PR не на чем.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file docker Docker image dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant