Skip to content

Workflow to test contract generated code#4336

Open
jmg-duarte wants to merge 5 commits intomainfrom
jmgd/fix/diff-gen-contracts
Open

Workflow to test contract generated code#4336
jmg-duarte wants to merge 5 commits intomainfrom
jmgd/fix/diff-gen-contracts

Conversation

@jmg-duarte
Copy link
Copy Markdown
Contributor

@jmg-duarte jmg-duarte commented Apr 15, 2026

Description

Adds a CI job to ensure that generated contract code was just generated

Changes

  • CI job to detect manual tampering of generated code

How to test

There are commits associated with this PR that should launch jobs that will fail or pass, will add them here as they're done

"Successful failure" — i.e. detected changes: https://github.com/cowprotocol/services/actions/runs/24463810679/job/71485172786?pr=4336
Successful success — i.e. no changes = nothing detected: https://github.com/cowprotocol/services/actions/runs/24464073112/job/71486114618?pr=4336

@jmg-duarte jmg-duarte requested a review from a team as a code owner April 15, 2026 15:41
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes a documentation comment from an auto-generated file. Feedback was provided to restore this comment to maintain consistency with the code generation template and prevent CI failures related to tampering detection.

Comment thread .github/workflows/pull-request.yaml Outdated
Comment thread .github/workflows/pull-request.yaml Outdated
Copy link
Copy Markdown
Member

@AryanGodara AryanGodara left a comment

Choose a reason for hiding this comment

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

left 2 comments

Comment thread .github/workflows/pull-request.yaml Outdated
Co-authored-by: Jan [Yann] <4518474+fafk@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@MartinquaXD MartinquaXD left a comment

Choose a reason for hiding this comment

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

Thank you.

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.

4 participants