Skip to content

Comments

[CERT-9836] Replace active reviews#73

Merged
H00N24 merged 8 commits intomainfrom
CERT-9836-reuse-active-reviews
Jan 28, 2026
Merged

[CERT-9836] Replace active reviews#73
H00N24 merged 8 commits intomainfrom
CERT-9836-reuse-active-reviews

Conversation

@H00N24
Copy link
Collaborator

@H00N24 H00N24 commented Jan 7, 2026

🚀 Pull Request Overview

This PR adds a new functionality to replace existing PR reviews instead of creating new ones.

Example:


📜 Tests Checklist

Before submitting this PR, ensure that all tests pass and meet the following conditions:

Category Status Compilation Comment Results Review
fail_to_start ❌ Failed with compilation error 🟢 Yes 🔴 No
violated_rules ✅ Passed, with violations found 🔴 No 🟢 Yes
verified_rules ✅ Passed without violations 🔴 No 🟢 Yes
solana_violated_rules ✅ Passed, with violations found 🔴 No 🟢 Yes
sui_verified_rules ✅ Passed without violations 🔴 No 🟢 Yes

Please verify that your changes meet the above conditions for each category of tests.
If something doesn't match, investigate before submitting the PR.


@H00N24 H00N24 requested a review from a team January 7, 2026 15:29
@github-actions
Copy link

github-actions bot commented Jan 7, 2026

Certora Run Started (Fail to Start)

  • Group ID: c62df8e4-30d0-4265-b2e8-cbeca37335bf
Config Status Link Log File
conf-start-error.conf Failed (1) - tests/evm/conf-start-error.conf-05e351f8b218.log
conf-start-error.conf --method "counter()" Failed (1) - tests/evm/conf-start-error.conf-3ae0eb7875e9.log
conf-verified.conf Compiled - tests/evm/conf-verified.conf-a263b9b4dee1.log
conf-verified.conf --rule monotone --method "counter()" Compiled - tests/evm/conf-verified.conf-09e19cf8319e.log
conf-violations.conf --method "counter()" Compiled - tests/evm/conf-violations.conf-48af602052c7.log
conf-violations.conf --rule invertible Compiled - tests/evm/conf-violations.conf-38305e633c04.log

Certora Run Summary

  • Started 4 jobs
  • 2 jobs failed

Download Logs

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 2b7f43a0-af10-40e7-a3f3-849e2def7cc5
Job Result VERIFIED VIOLATED Link
conf-violations.conf --rule invertible 0 0 Link
conf-violations.conf 2 1 Link
conf-verified.conf --method "counter()" 3 0 Link
conf-verified.conf 0 0 Link

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 43139c44-bed2-416f-88b6-8953adb854d7
  • Commit: aff1fb3
Job Result VERIFIED Link
conf-verified.conf --rule monotone --method "counter()" 2 Link
conf-verified.conf --rule invertible 2 Link
conf-verified.conf --method "counter()" 3 Link
conf-verified.conf 3 Link

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 26e98a00-fa46-4b1c-a500-b05cf274c504
  • Commit: 9766421
Job Result VERIFIED Link
conf-verified.conf --rule monotone --method "counter()" 2 Link
conf-verified.conf --rule invertible 2 Link
conf-verified.conf --method "counter()" 3 Link
conf-verified.conf 3 Link

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: fc4c8eb2-e93f-40e0-ace1-fbd1ec8a7310
Job Result VERIFIED Link
simple.conf 1 Link

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 5e7818b3-b064-4930-81e4-79b9da73444b
Job Result SANITY_FAILED VIOLATED Link
Default.conf 2 1 Link

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 5db210cd-e2d6-43c5-8006-e820d2c16a68
Job Result VERIFIED Link
Hello.conf 1 Link

@H00N24 H00N24 requested a review from ozcodes January 9, 2026 14:55
@H00N24 H00N24 enabled auto-merge (squash) January 21, 2026 12:45
Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 1f052ec2-c560-4897-8a0b-a7e9ee1ad8f0
  • Commit: 13d769e
Job Result VERIFIED VIOLATED Link
conf-violations.conf --rule invertible 2 0 Link
conf-violations.conf 2 1 Link
conf-verified.conf --method "counter()" 3 0 Link
conf-verified.conf 3 0 Link

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 7cf7767f-8b89-4d1e-905b-8908958942a8
  • Commit: 13d769e
Job Result SANITY_FAILED VIOLATED Link
Default.conf 2 1 Link

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 921d1772-486c-4e24-879b-58f2f687a690
  • Commit: 13d769e
Job Result VERIFIED Link
simple.conf 1 Link

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 299734ec-ee23-422b-8f7f-74cfbeca60e8
  • Commit: 13d769e
Job Result VERIFIED Link
Hello.conf 1 Link

* Cancel all jobs

* Add server

* Fix script path

* Export domain

* Add README

* Add error log
@H00N24 H00N24 force-pushed the CERT-9836-reuse-active-reviews branch from a4cf61c to 5018314 Compare January 21, 2026 13:12
Copy link
Collaborator

@ozcodes ozcodes left a comment

Choose a reason for hiding this comment

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

it looks good, i left a few questions

@H00N24 H00N24 requested a review from ozcodes January 26, 2026 12:12
@H00N24 H00N24 requested a review from ozcodes January 28, 2026 08:29
@H00N24 H00N24 merged commit 13d769e into main Jan 28, 2026
7 of 12 checks passed
@H00N24 H00N24 deleted the CERT-9836-reuse-active-reviews branch January 28, 2026 08:30
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