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

Fix for fabric renaming #8604

Merged
merged 3 commits into from
Jul 23, 2021
Merged

Conversation

mkardous-silabs
Copy link
Contributor

@mkardous-silabs mkardous-silabs commented Jul 23, 2021

Problem

What is being fixed? Examples:

  • Removed problematic header since it is not used anymore
  • Removed unused variable
    (relating to this commit 2a63309)

Change overview

Removed include
Removed variable

Testing

How was this tested? (at least one bullet point required)
Example app, python controller and linux standalone were locally built to validated change.

@CLAassistant
Copy link

CLAassistant commented Jul 23, 2021

CLA assistant check
All committers have signed the CLA.

@tcarmelveilleux tcarmelveilleux added the hotfix urgent fix needed, can bypass review label Jul 23, 2021
@boring-cyborg boring-cyborg bot added the app label Jul 23, 2021
@mkardous-silabs mkardous-silabs changed the title Fix for header file rename Fix for fabric renaming Jul 23, 2021
@yufengwangca yufengwangca merged commit 3838092 into project-chip:master Jul 23, 2021
nikita-s-wrk pushed a commit to nikita-s-wrk/connectedhomeip that referenced this pull request Sep 23, 2021
* Fix for header file rename

* Removed problematic include since it is not necessary anymore

* Removed unused variable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app hotfix urgent fix needed, can bypass review review - pending
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants