Skip to content

Fix NameError by using existing SIZE_2_8_INCH_NEWREV constant#1005

Open
wyldere wants to merge 1 commit into
mathoudebine:mainfrom
wyldere:fix-size-2-8-round-usb-constant
Open

Fix NameError by using existing SIZE_2_8_INCH_NEWREV constant#1005
wyldere wants to merge 1 commit into
mathoudebine:mainfrom
wyldere:fix-size-2-8-round-usb-constant

Conversation

@wyldere
Copy link
Copy Markdown

@wyldere wyldere commented May 17, 2026

Commit a3a375d (PR #995) referenced SIZE_2_8_ROUND_USB in five places
in configure.py but never defined the constant, causing a NameError at
import time. This adds the missing definition.

@wyldere wyldere force-pushed the fix-size-2-8-round-usb-constant branch from 500387e to bbd56c5 Compare May 17, 2026 16:04
@mathoudebine
Copy link
Copy Markdown
Owner

Sorry for that, the SIZE_2_8_ROUND_USB should instead be replaced by SIZE_2_8_INCH_NEWREV everywhere in the file

@wyldere wyldere force-pushed the fix-size-2-8-round-usb-constant branch from bbd56c5 to 81cf8da Compare May 23, 2026 00:30
@wyldere
Copy link
Copy Markdown
Author

wyldere commented May 23, 2026

Thanks for the review. Updated to use SIZE_2_8_INCH_NEWREV everywhere.

@wyldere wyldere changed the title Fix: define SIZE_2_8_ROUND_USB constant Fix NameError by using existing SIZE_2_8_INCH_NEWREV constant May 23, 2026
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