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

[nrf toup] [nrfconnect] remove ZMS_LOOKUP_CACHE_FOR_SETTINGS #557

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

Conversation

rghaddab
Copy link

@rghaddab rghaddab commented Feb 19, 2025

This config do not longer exist in sdk-zephyr

NCS PR: nrfconnect/sdk-nrf/pull/20495

@adigie
Copy link
Member

adigie commented Feb 19, 2025

@rghaddab could you amend the commit and force push? We need the workflow to pick up the change in PR description, so it can build with the correct NCS PR.

@rghaddab rghaddab force-pushed the rghaddab/remove-ZMS_LOOKUP_CACHE_FOR_SETTINGS branch from 90684fe to 2b20c79 Compare February 19, 2025 16:43
@rghaddab
Copy link
Author

@rghaddab could you amend the commit and force push? We need the workflow to pick up the change in PR description, so it can build with the correct NCS PR.

What do you I need to amend ?
The build is still failing

@adigie
Copy link
Member

adigie commented Feb 19, 2025

OK, it’s different issue then. I will take a look.

This config do no longer exist in sdk-zephyr

Signed-off-by: Riadh Ghaddab <[email protected]>
@adigie adigie force-pushed the rghaddab/remove-ZMS_LOOKUP_CACHE_FOR_SETTINGS branch from 0f17ca6 to f0c1631 Compare February 21, 2025 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants