Skip to content

Use python:3-slim base image - #16

Merged
solrbot merged 7 commits into
solrbot:mainfrom
janhoy:tirgger-8
Oct 25, 2025
Merged

solrbot merged 7 commits into
solrbot:mainfrom
janhoy:tirgger-8

Conversation

@janhoy

@janhoy janhoy commented Oct 25, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

Repository owner deleted a comment from github-actions Bot Oct 25, 2025
Repository owner deleted a comment from github-actions Bot Oct 25, 2025

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review with suggested changes

Comment thread CHANGELOG.md
- Legacy "I forgot the changelog" excuse handling (#445) (Raj Patel)
- Irrelevant multi-line entry that just discusses elephants and other completely
unrelated topics to the actual change in the PR. Let's see how Claude handles
this then (#111) (Jon Blow, Jane Snow)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This was converted to logchange format. Let's remove it.

Suggested change
this then (#111) (Jon Blow, Jane Snow)

@github-actions

Copy link
Copy Markdown

🔄 I've converted the legacy changelog entry to logchange format!

I detected a change to CHANGELOG.md and converted it to the logchange format below.

Suggested Logchange Entry for changelog/unreleased/pr-16-use-python-3-slim-base-image.yml:

title: Use python:3-slim base image
type: dependency_update
description: Updated Docker base image from python:3-slim AS builder to python:3-slim
authors:
  - name: Jon Blow
  - name: Jane Snow
issues:
  - 111

What to do next:

  1. ✅ Create the logchange entry:

    • Create a new file at changelog/unreleased/pr-16-use-python-3-slim-base-image.yml
    • Copy the YAML above into it
  2. ⚠️ Remove the legacy changes:

    • I've added suggested edits to remove the lines you added to CHANGELOG.md
    • Review the suggestions in the PR review and accept them
  3. 📝 Review before merging:

    • Check the generated entry is accurate
    • Adjust if needed
    • Feel free to edit the logchange YAML

Why?
This project uses logchange format for changelog entries, not the traditional CHANGELOG.md format. Using logchange ensures consistency and better tooling support.

@solrbot
solrbot merged commit 8fecc2b into solrbot:main Oct 25, 2025
2 of 4 checks 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