Commit bd3955a6 authored by smartcontracts's avatar smartcontracts Committed by GitHub

fix: add extra context to check interfaces script logs (#11861)

Updates the output of the check-interfaces script to be more clear
about cases when removing items from the EXCLUDE_CONTRACTS list is
acceptable and how to verify that an item is unnecessary.
parent 820636fb
...@@ -219,8 +219,9 @@ done ...@@ -219,8 +219,9 @@ done
# Check for unnecessary exclusions # Check for unnecessary exclusions
for exclude_item in "${EXCLUDE_CONTRACTS[@]}"; do for exclude_item in "${EXCLUDE_CONTRACTS[@]}"; do
if ! grep -q "^$exclude_item$" "$REPORTED_INTERFACES_FILE"; then if ! grep -q "^$exclude_item$" "$REPORTED_INTERFACES_FILE"; then
echo "Warning: $exclude_item is in the exclude list but was not reported as an issue." echo "Warning: $exclude_item is in the exclude list but WAS NOT reported as an issue. It"
echo "Consider removing it from the EXCLUDE_CONTRACTS list in the script." echo "may be unnecessary in the EXCLUDE_CONTRACTS list, but you MUST verify this before"
echo "removing it by performing a clean and full build before re-running this script."
fi fi
done done
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment