Skip to content

Conversation

@fbac
Copy link
Collaborator

@fbac fbac commented Aug 22, 2025

Close #913

I fixed this at some point, this was related to the way the PayerReport calculates its ID.

@graphite-app
Copy link

graphite-app bot commented Aug 22, 2025

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • Queue - adds this PR to the back of the merge queue
  • Hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

@fbac fbac marked this pull request as ready for review August 22, 2025 13:52
@fbac fbac requested a review from a team as a code owner August 22, 2025 13:52
@macroscopeapp
Copy link

macroscopeapp bot commented Aug 22, 2025

Enable execution of integration test TestCanGenerateAndAttestReport in integration_test.go

Remove the t.Skip call in integration_test.go.

📍Where to Start

Start with the TestCanGenerateAndAttestReport test in integration_test.go.


Macroscope summarized 35d7ce3.

Copy link
Collaborator

@mkysel mkysel left a comment

Choose a reason for hiding this comment

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

we will see whether it is really fixed

@fbac fbac merged commit 35799cd into main Aug 22, 2025
10 checks passed
@fbac fbac deleted the 08-22-GH913 branch August 22, 2025 13:57
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.

Test failure: TestCanGenerateAndAttestReport is flaky

3 participants