Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Arduino comments for post-release publish #8381

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

gojimmypi
Copy link
Contributor

@gojimmypi gojimmypi commented Jan 26, 2025

Description

This PR updates Arduino comments in various README and .ino sketch files to improve visibility to important wolfSSL user settings and configuration information.

NOTE: this is a post-release update to wolfSSL v5.7.6 that I plan to include in an upcoming published wolfSSL-Arduino v5.7.6

I'd like these comments to be included sooner than the next release due to increased Arduino questions related to custom configuration.

Fixes zd# 19275

Testing

Installed manually with ./wolfssl-arduino.sh INSTALL and tested with Arduino examples.

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@gojimmypi
Copy link
Contributor Author

Jenkins retest this please

@dgarske dgarske merged commit 8bf057c into wolfSSL:master Jan 27, 2025
159 checks passed
jmalak pushed a commit to jmalak/wolfssl that referenced this pull request Feb 5, 2025
…c80-arduino

Update Arduino comments for post-release publish
jmalak pushed a commit to jmalak/wolfssl that referenced this pull request Feb 6, 2025
…c80-arduino

Update Arduino comments for post-release publish
jmalak pushed a commit to jmalak/wolfssl that referenced this pull request Feb 6, 2025
…c80-arduino

Update Arduino comments for post-release publish
jmalak pushed a commit to jmalak/wolfssl that referenced this pull request Feb 23, 2025
…c80-arduino

Update Arduino comments for post-release publish
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.

3 participants