Skip to content

Conversation

danyalprout
Copy link
Collaborator

@danyalprout danyalprout commented Oct 8, 2025

Description

Update the formatting of the message. Commit message will look like:

    chore: updated node-reth, nethermind, op-geth, optimism

    ### Dependency Updates
    **node-reth** - v0.1.9:  [diff](https://github.com/base/node-reth/compare/v0.1.8...v0.1.9)
    **nethermind** - 1.34.1:  [diff](https://github.com/NethermindEth/nethermind/compare/1.34.0...1.34.1)
    **op-geth** - v1.101603.1:  [diff](https://github.com/ethereum-optimism/op-geth/compare/v1.101602.3...v1.101603.1)
    **optimism** - op-node/v1.14.1:  [diff](https://github.com/ethereum-optimism/optimism/compare/op-node/v1.13.7...op-node/v1.14.1)```

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Oct 8, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@danyalprout
Copy link
Collaborator Author

danyalprout commented Oct 8, 2025

Formatted example:

Dependency Updates

node-reth - v0.1.9: diff
nethermind - 1.34.1: diff
op-geth - v1.101603.1: diff
optimism - op-node/v1.14.1: diff

repos = append(repos, repo)
}
commitDescription = strings.TrimSuffix(commitDescription, " ")
commitDescription := strings.Join(descriptionLines, "\n")
Copy link
Collaborator

Choose a reason for hiding this comment

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

While you're at it - can you set the type / risk / severity in the PR description? Or do you want a human to set those before merging (looks required by infra)

@danyalprout danyalprout merged commit db71d5f into main Oct 8, 2025
11 checks passed
@danyalprout danyalprout deleted the formatting-pr branch October 8, 2025 22:52
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.

3 participants