- 
                Notifications
    You must be signed in to change notification settings 
- Fork 0
Prepare for release #9
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
Conversation
| Memory usage change @ 3fd301c 
 Click for full report table
 Click for full report CSV | 
| Codecov ReportAll modified and coverable lines are covered by tests ✅ 
 Additional details and impacted files@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage   75.98%   75.98%           
=======================================
  Files           4        4           
  Lines         304      304           
=======================================
  Hits          231      231           
  Misses         73       73           ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
 | 
| Memory usage change @ 9792dca 
 Click for full report table
 Click for full report CSV | 
9792dca    to
    4c6c46e      
    Compare
  
    | Memory usage change @ 4c6c46e 
 Click for full report table
 Click for full report CSV | 
There was a problem hiding this 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 prepares the release by updating include paths to use the new connectionHandlerModels locations instead of the legacy settings directories. The key changes include:
- Replacing all instances of "#include <settings/…>" with "#include <connectionHandlerModels/settings…>".
- Removing obsolete include directives that reference settings_default from files where they're no longer needed.
- Updating the GitHub workflow file to use package names rather than URLs for dependency resolution.
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description | 
|---|---|
| src/configuratorAgents/agents/boardConfigurationProtocol/cbor/ProvisioningMessage.h | Updated include path for settings | 
| src/configuratorAgents/agents/boardConfigurationProtocol/cbor/Decoder.cpp | Updated include path for settings_default | 
| src/configuratorAgents/agents/boardConfigurationProtocol/CBORAdapter.h | Updated include path for settings | 
| src/configuratorAgents/agents/boardConfigurationProtocol/CBORAdapter.cpp | Removed redundant include for settings_default | 
| src/configuratorAgents/agents/ConfiguratorAgent.h | Removed obsolete settings include | 
| src/configuratorAgents/MessagesDefinitions.h | Updated include path for settings | 
| src/configuratorAgents/AgentsManager.cpp | Removed obsolete settings include | 
| src/Arduino_NetworkConfigurator.h | Updated include path for settings | 
| README.md | Updated example code and documentation to reflect changes | 
| .github/workflows/compile-examples.yml | Dependencies are now referenced by name | 
4c6c46e    to
    2c1fd9c      
    Compare
  
    | Memory usage change @ 2c1fd9c 
 Click for full report table
 Click for full report CSV | 
| Memory usage change @ 5a27b1a 
 Click for full report table
 Click for full report CSV | 
5a27b1a    to
    2c1fd9c      
    Compare
  
    
No description provided.