Skip to content

Avoid stale entries in Workflow Registry#273

Merged
ibrajer merged 5 commits into
developfrom
workflow-registry-stale-entries
Oct 15, 2025
Merged

Avoid stale entries in Workflow Registry#273
ibrajer merged 5 commits into
developfrom
workflow-registry-stale-entries

Conversation

@ibrajer
Copy link
Copy Markdown
Contributor

@ibrajer ibrajer commented Oct 13, 2025

No description provided.

@ibrajer ibrajer self-assigned this Oct 13, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 13, 2025

Static analysis results are available

Hey @ibrajer, you can view Slither reports in the job summary here or download them as artifact here.
Please check them before merging and make sure you have addressed all issues.

@ibrajer ibrajer force-pushed the workflow-registry-stale-entries branch from 89e80bf to f87103d Compare October 13, 2025 12:33
vm.stopPrank();
}

function test_upsertWorkflow_WhenDonFamilyIsChangedOnUpdateWorkflow()
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
function test_upsertWorkflow_WhenDonFamilyIsChangedOnUpdateWorkflow()
function test_RevertWhen_upsertWorkflow_WhenDonFamilyIsChangedOnUpdateWorkflow()

Tests that are expected to revert must have a _RevertWhen_ affix as per the Solidity style guide.

DeividasK
DeividasK previously approved these changes Oct 14, 2025
@ibrajer ibrajer marked this pull request as ready for review October 14, 2025 10:16
@ibrajer ibrajer requested a review from a team as a code owner October 14, 2025 10:16
Atrax1
Atrax1 previously approved these changes Oct 14, 2025
@Atrax1 Atrax1 requested a review from a team October 14, 2025 10:17
infiloop2
infiloop2 previously approved these changes Oct 14, 2025
Copy link
Copy Markdown
Contributor

@infiloop2 infiloop2 left a comment

Choose a reason for hiding this comment

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

lgtm

@ibrajer ibrajer dismissed stale reviews from infiloop2, Atrax1, and DeividasK via 22157a7 October 14, 2025 15:34
@ibrajer ibrajer force-pushed the workflow-registry-stale-entries branch from 22157a7 to b911076 Compare October 14, 2025 16:41
@ibrajer ibrajer merged commit 729ba0b into develop Oct 15, 2025
53 checks passed
@ibrajer ibrajer deleted the workflow-registry-stale-entries branch October 15, 2025 11:55
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