Skip to content

Commit 2e2b4db

Browse files
authored
Add summary to verify script (#3309)
* Add verification check for killed chains * Add killed chain to summary, simplify high liquidity table 1. Moved Killed Chain Assets Section - Now appears as a subsection at the bottom of the "Failed Checks" analysis section (after Logo Failures) - Always displays with fallback text when there are no killed chain assets: "No assets from killed chains detected. All assets belong to active chains." - Follows the same pattern as other report subsections 2. Merged High Liquidity Table Columns - Combined "Bid Depth" and "Other Failures" into a single "Failure Reasons" column - Bid depth failures now show as "Bid depth: [details]" within the merged column - All failures are now listed together, comma-separated, for easier reading * Update check_verification_criteria.yml
1 parent 9f99f24 commit 2e2b4db

File tree

1 file changed

+19
-0
lines changed

1 file changed

+19
-0
lines changed

.github/workflows/check_verification_criteria.yml

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,10 +53,12 @@ jobs:
5353
5454
READY=$(jq -r '.summary.readyForVerification' "$REPORT_PATH")
5555
FAILED=$(jq -r '.summary.failedChecks' "$REPORT_PATH")
56+
KILLED=$(jq -r '.summary.killedChainAssets' "$REPORT_PATH")
5657
TOTAL=$(jq -r '.summary.totalChecked' "$REPORT_PATH")
5758
5859
echo "- ✅ **Ready for Verification**: $READY" >> $GITHUB_STEP_SUMMARY
5960
echo "- ❌ **Failed Checks**: $FAILED" >> $GITHUB_STEP_SUMMARY
61+
echo "- ⚠️ **Killed Chain Assets**: $KILLED" >> $GITHUB_STEP_SUMMARY
6062
echo "- 📊 **Total Checked**: $TOTAL" >> $GITHUB_STEP_SUMMARY
6163
echo "" >> $GITHUB_STEP_SUMMARY
6264
@@ -157,6 +159,23 @@ jobs:
157159
fi
158160
echo "" >> $GITHUB_STEP_SUMMARY
159161
162+
# Killed Chain Assets
163+
echo "### ⚠️ Verified Assets on Killed Chains" >> $GITHUB_STEP_SUMMARY
164+
echo "" >> $GITHUB_STEP_SUMMARY
165+
echo "Verified assets belonging to chains marked as 'killed' in the chain registry (excluding meme tokens):" >> $GITHUB_STEP_SUMMARY
166+
echo "" >> $GITHUB_STEP_SUMMARY
167+
KILLED_CHAIN_COUNT=$(jq -r '.summary.killedChainAssets' "$REPORT_PATH")
168+
if [ "$KILLED_CHAIN_COUNT" -gt 0 ]; then
169+
echo "| Asset | Chain | Base Denom |" >> $GITHUB_STEP_SUMMARY
170+
echo "|-------|-------|------------|" >> $GITHUB_STEP_SUMMARY
171+
jq -r '.analysis.killedChainAssets[] | "| \(.comment // .base_denom) | \(.chain_name) | `\(.base_denom)` |"' "$REPORT_PATH" >> $GITHUB_STEP_SUMMARY
172+
echo "" >> $GITHUB_STEP_SUMMARY
173+
echo "_**Note:** Meme tokens are exempt from this requirement and may remain verified on killed chains._" >> $GITHUB_STEP_SUMMARY
174+
else
175+
echo "_No verified assets from killed chains detected. All verified assets belong to active chains._" >> $GITHUB_STEP_SUMMARY
176+
fi
177+
echo "" >> $GITHUB_STEP_SUMMARY
178+
160179
echo "---" >> $GITHUB_STEP_SUMMARY
161180
echo "" >> $GITHUB_STEP_SUMMARY
162181
echo "📥 **Download the full JSON report** from the workflow artifacts for complete details on all checks." >> $GITHUB_STEP_SUMMARY

0 commit comments

Comments
 (0)