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

Integrate ZAP into sample application CI #1733

Closed
franck-apple opened this issue Jul 23, 2020 · 2 comments
Closed

Integrate ZAP into sample application CI #1733

franck-apple opened this issue Jul 23, 2020 · 2 comments
Assignees
Labels
p1 priority 1 work V1.X zap
Milestone

Comments

@franck-apple
Copy link

franck-apple commented Jul 23, 2020

Problem

Replace the files inside /gen/ folder with file generated from the ZAP application

Proposed Solution

  1. Starting with the data available inside the ZCL xml, generate only the required files
  2. Move all required functions already present inside the /gen/ folders of examples application to /src/app/
  3. Cleanup all unnecessary data from the generated files.
  4. Add a generic .zap file to each /gen/ folder in order to allow regeneration of the generated files
@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label feature_request to this issue, with a confidence of 0.56. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@woody-apple woody-apple added this to the V1.0 milestone Jul 31, 2020
@wehale wehale changed the title Build ZAP CHIP Reference Plugin set in CI Integrate ZAP into sample application CI Sep 22, 2020
@franck-apple franck-apple modified the milestones: V1.0, 2020-10-30 Oct 7, 2020
@franck-apple franck-apple added the p1 priority 1 work label Oct 16, 2020
@jepenven-silabs jepenven-silabs self-assigned this Oct 21, 2020
@woody-apple woody-apple modified the milestones: 2020-10-30, V0.7 Jan 29, 2021
@woody-apple woody-apple added V1.X and removed V0.7 labels Jan 25, 2022
@vivien-apple
Copy link
Contributor

I assume this task has been done. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p1 priority 1 work V1.X zap
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants