Skip to content

Conversation

@Legoclones
Copy link

I went through and cleaned up some of the documentation/code comment spelling, capitalization, and grammar. There's also a bug fix or two sprinkled in there that I saw along the way, but hopefully it's pretty straightforward.

Legoclones added 2 commits May 9, 2025 19:38
Mostly spelling/capitalization, a few small formatting changes also. Almost all changes are in documentation/code comments
@alex-dewar
Copy link
Contributor

Hi,

Thanks for the pull request. 2 things:

  1. The bug fix you mentioned seems to be in an unused class, so it will probably get removed at some point
  2. We'd prefer to have the shorthand for capture boards without the hyphen (so CWLite, CWHusky, etc.)

Other than that, everything looks good

@Legoclones
Copy link
Author

Okay, I changed all CW-<product to CW<product> that I could find in the code. Just to be clear, do you prefer ChipWhisperer-Lite or ChipWhisperer Lite?

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