Skip to content

Add example for erasing the network configuration via library api #20

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fabik111
Copy link
Collaborator

@fabik111 fabik111 commented Aug 8, 2025

No description provided.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a new example sketch demonstrating how to programmatically delete/reset network configuration using the NetworkConfigurator library API. The example provides a simple utility that wipes stored network configuration without using the reconfiguration procedure.

Key Changes

  • Added a new DeleteConfiguration.ino example sketch with basic network configuration reset functionality
  • Updated CI workflow to include the new example in compilation testing

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
examples/utility/DeleteConfiguration/DeleteConfiguration.ino New example sketch that demonstrates network configuration deletion using the NetworkConfigurator library
.github/workflows/compile-examples.yml Added the new example to the CI compilation workflow

void setup() {
/* Initialize serial debug port and wait up to 5 seconds for port to open */
Serial.begin(9600);
for(unsigned long const serialBeginTime = millis(); !Serial && (millis() - serialBeginTime <= 5000); ) { }
Copy link
Preview

Copilot AI Aug 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The empty for loop body should include a comment or explicit empty block to clarify the intentional waiting behavior, e.g., { /* Wait for Serial */ }

Suggested change
for(unsigned long const serialBeginTime = millis(); !Serial && (millis() - serialBeginTime <= 5000); ) { }
for(unsigned long const serialBeginTime = millis(); !Serial && (millis() - serialBeginTime <= 5000); ) { /* Wait for Serial */ }

Copilot uses AI. Check for mistakes.

Copy link

github-actions bot commented Aug 8, 2025

Memory usage change @ 8440f09

Board flash % RAM for global variables %
arduino:mbed_giga:giga 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:mbed_nano:nanorp2040connect 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:mbed_nicla:nicla_vision 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:mbed_opta:opta 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:mbed_portenta:envie_m7 N/A N/A N/A N/A
arduino:renesas_portenta:portenta_c33 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:renesas_uno:unor4wifi 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:samd:mkrwifi1010 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
arduino:samd:nano_33_iot 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
Click for full report table
Board examples/NetworkConfiguratorDemo
flash
% examples/NetworkConfiguratorDemo
RAM for global variables
% examples/utility/DeleteConfiguration
flash
% examples/utility/DeleteConfiguration
RAM for global variables
%
arduino:mbed_giga:giga 0 0.0 0 0.0 N/A N/A N/A N/A
arduino:mbed_nano:nanorp2040connect 0 0.0 0 0.0 N/A N/A N/A N/A
arduino:mbed_nicla:nicla_vision 0 0.0 0 0.0 N/A N/A N/A N/A
arduino:mbed_opta:opta 0 0.0 0 0.0 N/A N/A N/A N/A
arduino:mbed_portenta:envie_m7 N/A N/A N/A N/A N/A N/A N/A N/A
arduino:renesas_portenta:portenta_c33 0 0.0 0 0.0 N/A N/A N/A N/A
arduino:renesas_uno:unor4wifi 0 0.0 0 0.0 N/A N/A N/A N/A
arduino:samd:mkrwifi1010 0 0.0 0 0.0 N/A N/A N/A N/A
arduino:samd:nano_33_iot 0 0.0 0 0.0 N/A N/A N/A N/A
Click for full report CSV
Board,examples/NetworkConfiguratorDemo<br>flash,%,examples/NetworkConfiguratorDemo<br>RAM for global variables,%,examples/utility/DeleteConfiguration<br>flash,%,examples/utility/DeleteConfiguration<br>RAM for global variables,%
arduino:mbed_giga:giga,0,0.0,0,0.0,N/A,N/A,N/A,N/A
arduino:mbed_nano:nanorp2040connect,0,0.0,0,0.0,N/A,N/A,N/A,N/A
arduino:mbed_nicla:nicla_vision,0,0.0,0,0.0,N/A,N/A,N/A,N/A
arduino:mbed_opta:opta,0,0.0,0,0.0,N/A,N/A,N/A,N/A
arduino:mbed_portenta:envie_m7,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A
arduino:renesas_portenta:portenta_c33,0,0.0,0,0.0,N/A,N/A,N/A,N/A
arduino:renesas_uno:unor4wifi,0,0.0,0,0.0,N/A,N/A,N/A,N/A
arduino:samd:mkrwifi1010,0,0.0,0,0.0,N/A,N/A,N/A,N/A
arduino:samd:nano_33_iot,0,0.0,0,0.0,N/A,N/A,N/A,N/A

@per1234 per1234 added topic: documentation Related to documentation for the project type: enhancement Proposed improvement labels Aug 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: documentation Related to documentation for the project type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants