Skip to content

Update heylogs to 0.18.0 and use github-actions format#15786

Merged
koppor merged 1 commit into
mainfrom
update-heylogs
May 20, 2026
Merged

Update heylogs to 0.18.0 and use github-actions format#15786
koppor merged 1 commit into
mainfrom
update-heylogs

Conversation

@koppor

@koppor koppor commented May 20, 2026

Copy link
Copy Markdown
Member

Related issues and pull requests

None.

PR Description

Bumps the heylogs CLI used by the CHANGELOG.md lint job in tests-code.yml from 0.17.1 to 0.18.0 and switches it to the new --format github-actions output so changelog issues now show up as inline PR annotations on the offending line and column instead of being buried in the workflow log. The previous grep -q "1 problem" summary-string check is replaced with a count of ::error annotations, since the new format emits no summary line; the inline comment is also corrected from "two acceptable errors" to "one acceptable error" to match the actual gate.

Steps to test

  1. Open the PR and confirm the tests-code.yml → CHANGELOG.md job passes.

  2. In the PR "Files changed" view of CHANGELOG.md, confirm the pre-existing acceptable error appears as an inline annotation at CHANGELOG.md:2035:1 (Missing ref link (all-h2-contain-a-version)).

  3. Optional local check:

    jbang com.github.nbbrd.heylogs:heylogs-cli:0.18.0:bin check --format github-actions CHANGELOG.md

    Should print exactly one line:

    ::error file=CHANGELOG.md,line=2035,col=1::Missing ref link (all-h2-contain-a-version)

  4. To verify the gate still fails on regressions, temporarily introduce an extra changelog issue locally and confirm error_count != 1 causes the workflow step to exit non-zero.

AI usage

Claude Code (model claude-opus-4-7)

Checklist

  • I own the copyright of the code submitted and I license it under the MIT license
  • If AI tools were used, I disclosed them in the "AI usage" section and reviewed, understood, and take full ownership of all AI-generated code
  • [/] I manually tested my changes in running JabRef (always required)
  • [/] I added JUnit tests for changes (if applicable)
  • [/] I added screenshots in the PR description (if change is visible to the user)
  • [/] I added a screenshot in the PR description showing a library with a single entry with me as author and as title the issue number
  • [/] I described the change in CHANGELOG.md in a way that can be understood by the average user (if change is visible to the user)
  • [/] I checked the user documentation for up to dateness and submitted a pull request to our user documentation repository

@koppor @claude
Update heylogs to 0.18.0 and use github-actions format
e0d3c9b 
Switches the heylogs CLI invocation in tests-code.yml to 0.18.0 and
uses the new --format github-actions output so changelog issues appear
as inline PR annotations on the offending lines.

Replaces the previous "1 problem" summary-string check with a count of
::error annotations since the github-actions format does not emit a
summary line.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented May 20, 2026

Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Upgrade heylogs to 0.18.0 with github-actions format

✨ Enhancement

Grey Divider

Walkthroughs

Description 
• Upgrade heylogs CLI from 0.17.1 to 0.18.0
• Switch to github-actions format for inline PR annotations
• Replace summary-string check with error count validation
• Correct comment from "two acceptable errors" to "one"
Diagram 
flowchart LR
  A["heylogs 0.17.1<br/>grep summary check"] -->|upgrade & format| B["heylogs 0.18.0<br/>github-actions format"]
  B -->|inline annotations| C["PR file changes view<br/>with error markers"]
  B -->|error count validation| D["Improved gate logic"]
Loading

Grey Divider

File Changes

1. .github/workflows/tests-code.yml ⚙️ Configuration changes +5/-4

Update heylogs version and output format

• Bumped heylogs CLI version from 0.17.1 to 0.18.0
• Added --format github-actions flag to enable inline PR annotations
• Replaced grep -q "1 problem" check with grep -c '^::error' for error counting
• Updated comment to reflect one acceptable error instead of two

.github/workflows/tests-code.yml

Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented May 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@koppor koppor added the automerge PR is tagged with that label will be merged if workflows are green label May 20, 2026
@koppor koppor enabled auto-merge May 20, 2026 07:56
@koppor koppor added this pull request to the merge queue May 20, 2026
@github-actions github-actions Bot added the status: to-be-merged PRs which are accepted and should go into the merge-queue. label May 20, 2026
Merged via the queue into main with commit 87eb852 May 20, 2026
68 of 72 checks passed
@koppor koppor deleted the update-heylogs branch May 20, 2026 08:35
Siedlerchr added a commit that referenced this pull request May 20, 2026
@Siedlerchr
Merge remote-tracking branch 'upstream/main' into performanceDuplicate
d29ddec 
* upstream/main:
  Update PULL_REQUEST_TEMPLATE.md (#15788)
  New Crowdin updates (#15787)
  Update heylogs to 0.18.0 and use github-actions format (#15786)
  Grand refactoring of the AI features (#15688)
  Chore(deps): Bump com.fasterxml:aalto-xml in /versions (#15782)
  Chore(deps): Bump org.junit:junit-bom from 6.0.3 to 6.1.0 in /versions (#15783)
  Fix default value for unwanted characters (#15743)
  Fix runner tag
  Fix runner for JBang (PR)
  Fix duplicate finder progress counter incrementing on empty queue polls (#15781)
  Refine JabKit CLI: positional input argument and check command group (#15759)
  Ignore exception in unregisterListener to prevent exception (#15761)
  Fix wrong usage of "key" (#15779)
  Fix Hayagriva export to nest identifiers under serial-number (#15750)
f0restron07 pushed a commit to f0restron07/jabref that referenced this pull request May 24, 2026
@koppor @claude @f0restron07
Update heylogs to 0.18.0 and use github-actions format (JabRef#15786)
da42a9e 
Switches the heylogs CLI invocation in tests-code.yml to 0.18.0 and
uses the new --format github-actions output so changelog issues appear
as inline PR annotations on the offending lines.

Replaces the previous "1 problem" summary-string check with a count of
::error annotations since the github-actions format does not emit a
summary line.

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
@koppor koppor mentioned this pull request May 26, 2026
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jabref-machine jabref-machine jabref-machine approved these changes

Assignees

No one assigned

Labels

automerge PR is tagged with that label will be merged if workflows are green status: to-be-merged PRs which are accepted and should go into the merge-queue.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@koppor @jabref-machine