Skip to content

Conversation

@janhoy
Copy link
Collaborator

@janhoy janhoy commented Oct 25, 2025

No description provided.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Review with suggested changes

- GitHub Actions integration that blocks your PR until changelog is perfect (#489) (Dana Torres)
- 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)

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)

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Review with suggested changes

unrelated topics to the actual change in the PR. Let's see how Claude handles
this then (#111) (Jon Blow, Jane Snow)


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

@github-actions
Copy link

🔄 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-17-advanced-changelog-edit.yml:

title: Advanced changelog edit
type: other
description: >
  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
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-17-advanced-changelog-edit.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 c0e92c2 into solrbot:main Oct 26, 2025
1 check 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