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

CHIP QRCode command line tool implementation #770

Merged
merged 4 commits into from
May 20, 2020

Conversation

shana-apple
Copy link
Contributor

Problem

Vendors need a command line tool to help generate qr codes from setup payload

Create a C++ QRCode command line tool

Summary of Changes

Create a command line tool that can generate a qr code or manual code from a text file.

fixes #165

src/Makefile.am Outdated Show resolved Hide resolved
Copy link
Contributor

@woody-apple woody-apple left a comment

Choose a reason for hiding this comment

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

@shana-apple please fix the build :)

Copy link
Contributor

@bhaskar-apple bhaskar-apple left a comment

Choose a reason for hiding this comment

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

LGTM

configure.ac Outdated Show resolved Hide resolved
src/Makefile.am Outdated Show resolved Hide resolved
Copy link

@hawk248 hawk248 left a comment

Choose a reason for hiding this comment

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

👍

src/chipcli/Makefile.am Outdated Show resolved Hide resolved
@bhaskar-apple bhaskar-apple force-pushed the sazria/cltool branch 2 times, most recently from def49ee to ae87515 Compare May 20, 2020 00:05
@codecov
Copy link

codecov bot commented May 20, 2020

Codecov Report

Merging #770 into master will decrease coverage by 3.14%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #770      +/-   ##
==========================================
- Coverage   59.16%   56.01%   -3.15%     
==========================================
  Files         130      133       +3     
  Lines       12652    12836     +184     
==========================================
- Hits         7485     7190     -295     
- Misses       5167     5646     +479     
Impacted Files Coverage Δ
src/qrcodetool/qrcodetool.cpp 0.00% <0.00%> (ø)
src/qrcodetool/setup_payload_commands.cpp 0.00% <0.00%> (ø)
src/setup_payload/SetupPayloadHelper.cpp 0.00% <0.00%> (ø)
src/ble/BleError.cpp 0.00% <0.00%> (-99.05%) ⬇️
src/inet/InetLayerBasis.cpp 0.00% <0.00%> (-80.00%) ⬇️
src/inet/AsyncDNSResolverSockets.cpp 0.00% <0.00%> (-46.96%) ⬇️
src/system/SystemPacketBuffer.cpp 49.71% <0.00%> (-46.33%) ⬇️
src/lib/core/CHIPTLVTags.h 66.66% <0.00%> (-33.34%) ⬇️
src/system/SystemLayer.cpp 55.47% <0.00%> (-20.55%) ⬇️
src/inet/IPAddress.cpp 0.00% <0.00%> (-18.25%) ⬇️
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 237f570...1c3ae79. Read the comment docs.

@woody-apple
Copy link
Contributor

@saurabhst ?

@woody-apple woody-apple merged commit ab8ffd5 into project-chip:master May 20, 2020
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.

Need a CL tool to generate QR code payload
10 participants