Skip to content

Remove app snippets - #25

Closed
ArekBalysNordic wants to merge 2 commits into
mainfrom
remove_app_snippets
Closed

ArekBalysNordic wants to merge 2 commits into
mainfrom
remove_app_snippets

Conversation

@ArekBalysNordic

Copy link
Copy Markdown
Owner

No description provided.

@@ -8,4 +8,4 @@
CONFIG_CHIP_ICD_LIT_SUPPORT=y
CONFIG_CHIP_ICD_SIT_SLOW_POLL_LIMIT=5000
CONFIG_CHIP_ICD_REPORT_ON_ACTIVE_MODE=y
CONFIG_NCS_SAMPLE_MATTER_ZAP_FILE_PATH="${APPLICATION_CONFIG_DIR}/snippets/lit_icd/light_switch.zap"
CONFIG_NCS_SAMPLE_MATTER_ZAP_FILE_PATH="${APPLICATION_CONFIG_DIR}/src/lit_icd_zap/light_switch.zap"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think we agreed that light switch is a device type that could be LIT by default, so I think we could just enable it and do not maintain two options.

Changed application snippets to overlay.conf files.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
- Converted sample-specific snippets to overlays.
- Moved common snippets to the global snippet directory.
- Aligned documentation.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
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