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 rpc_console README indentation #18980

Merged
merged 1 commit into from
Jun 10, 2022

Conversation

cpagravel
Copy link
Contributor

Problem

The README for rpc_console has incorrect indentation for intended rendering. It looks like this was changed with #17335 in an attempt to affect how code blocks in readmes were parsed.

Change overview

Changed indentation in examples/common/pigweed/rpc_console/README.md

@github-actions
Copy link

github-actions bot commented May 31, 2022

PR #18980: Size comparison from 6ef806b to 2bd6c99

Full report (41 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 6ef806b 2bd6c99 change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 648263 648263 0 0.0
(read/write) 158900 158900 0 0.0
.bss 74580 74580 0 0.0
.data 3404 3404 0 0.0
.rodata 84263 84263 0 0.0
.text 563764 563764 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 615103 615103 0 0.0
(read/write) 158072 158072 0 0.0
.bss 73812 73812 0 0.0
.data 3344 3344 0 0.0
.rodata 73655 73655 0 0.0
.text 541212 541212 0 0.0
lock-ftd LP_CC2652R7 (read only) 679815 679815 0 0.0
(read/write) 163008 163008 0 0.0
.bss 72604 72604 0 0.0
.data 3268 3268 0 0.0
.rodata 96199 96199 0 0.0
.text 583132 583132 0 0.0
lock-mtd LP_CC2652R7 (read only) 629223 629223 0 0.0
(read/write) 145724 145724 0 0.0
.bss 68340 68340 0 0.0
.data 3268 3268 0 0.0
.rodata 96087 96087 0 0.0
.text 532644 532644 0 0.0
pump-app LP_CC2652R7 (read only) 676219 676219 0 0.0
(read/write) 167772 167772 0 0.0
.bss 73004 73004 0 0.0
.data 3304 3304 0 0.0
.rodata 88675 88675 0 0.0
.text 587060 587060 0 0.0
pump-controller-app LP_CC2652R7 (read only) 654163 654163 0 0.0
(read/write) 189684 189684 0 0.0
.bss 72860 72860 0 0.0
.data 3264 3264 0 0.0
.rodata 83419 83419 0 0.0
.text 570264 570264 0 0.0
shell LP_CC2652R7 (read only) 641262 641262 0 0.0
(read/write) 154456 154456 0 0.0
.bss 76932 76932 0 0.0
.data 3408 3408 0 0.0
.rodata 81238 81238 0 0.0
.text 559792 559792 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 621842 621842 0 0.0
.app_xip_area 525420 525420 0 0.0
.bss 79064 79064 0 0.0
.data 708 708 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 630550 630550 0 0.0
.app_xip_area 535600 535600 0 0.0
.bss 77624 77624 0 0.0
.data 676 676 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 570902 570902 0 0.0
.app_xip_area 466024 466024 0 0.0
.bss 87224 87224 0 0.0
.data 616 616 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 914752 914752 0 0.0
(read/write) 133184 133184 0 0.0
.bss 131088 131088 0 0.0
.data 2096 2096 0 0.0
.text 914744 914744 0 0.0
BRD4161A+rpc (read only) 948988 948988 0 0.0
(read/write) 149876 149876 0 0.0
.bss 147576 147576 0 0.0
.data 2300 2300 0 0.0
.text 948980 948980 0 0.0
BRD4161A+rs911x (read only) 789196 789196 0 0.0
(read/write) 129460 129460 0 0.0
.bss 127356 127356 0 0.0
.data 2104 2104 0 0.0
.text 789188 789188 0 0.0
lock-app BRD4161A+wf200 (read only) 954352 954352 0 0.0
(read/write) 128244 128244 0 0.0
.bss 126172 126172 0 0.0
.data 2068 2068 0 0.0
.text 954344 954344 0 0.0
window-app BRD4161A (read only) 899680 899680 0 0.0
(read/write) 133256 133256 0 0.0
.bss 131160 131160 0 0.0
.data 2092 2092 0 0.0
.text 899672 899672 0 0.0
esp32 all-clusters-app c3devkit (read only) 1005432 1005432 0 0.0
(read/write) 1479594 1479594 0 0.0
.dram0.bss 69144 69144 0 0.0
.dram0.data 14656 14656 0 0.0
.flash.rodata 210360 210360 0 0.0
.flash.text 1005432 1005432 0 0.0
.iram0.text 62954 62954 0 0.0
m5stack (read only) 1060587 1060587 0 0.0
(read/write) 481808 481808 0 0.0
.dram0.bss 74664 74664 0 0.0
.dram0.data 34208 34208 0 0.0
.flash.rodata 240940 240940 0 0.0
.flash.text 1055203 1055203 0 0.0
.iram0.text 123267 123267 0 0.0
k32w light k32w061+release (read/write) 681000 681000 0 0.0
.bss 80168 80168 0 0.0
.data 2012 2012 0 0.0
.text 597116 597116 0 0.0
lock k32w061+release (read/write) 732504 732504 0 0.0
.bss 80600 80600 0 0.0
.data 1980 1980 0 0.0
.text 648220 648220 0 0.0
linux all-clusters-app debug (read only) 2769441 2769441 0 0.0
(read/write) 178272 178272 0 0.0
.bss 86240 86240 0 0.0
.data 2032 2032 0 0.0
.data.rel.ro 83832 83832 0 0.0
.dynamic 608 608 0 0.0
.got 4496 4496 0 0.0
.init 27 27 0 0.0
.init_array 1016 1016 0 0.0
.rodata 243677 243677 0 0.0
.text 2351858 2351858 0 0.0
all-clusters-minimal-app debug (read only) 2646561 2646561 0 0.0
(read/write) 170272 170272 0 0.0
.bss 85504 85504 0 0.0
.data 1904 1904 0 0.0
.data.rel.ro 76712 76712 0 0.0
.dynamic 608 608 0 0.0
.got 4496 4496 0 0.0
.init 27 27 0 0.0
.init_array 1016 1016 0 0.0
.rodata 245597 245597 0 0.0
.text 2228450 2228450 0 0.0
bridge-app debug+rpc (read only) 2029753 2029753 0 0.0
(read/write) 147896 147896 0 0.0
.bss 72896 72896 0 0.0
.data 3936 3936 0 0.0
.data.rel.ro 65480 65480 0 0.0
.dynamic 592 592 0 0.0
.got 4272 4272 0 0.0
.init 27 27 0 0.0
.init_array 688 688 0 0.0
.rodata 168768 168768 0 0.0
.text 1704514 1704514 0 0.0
chip-tool debug (read only) 9680645 9680645 0 0.0
(read/write) 602736 602736 0 0.0
.bss 23968 23968 0 0.0
.data 1120 1120 0 0.0
.data.rel.ro 571336 571336 0 0.0
.dynamic 624 624 0 0.0
.got 5008 5008 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 496317 496317 0 0.0
.text 7790965 7790965 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 9423884 9423884 0 0.0
(read/write) 668993 668993 0 0.0
.bss 42257 42257 0 0.0
.data 1176 1176 0 0.0
.data.rel.ro 606736 606736 0 0.0
.dynamic 528 528 0 0.0
.got 15008 15008 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 460036 460036 0 0.0
.text 7435140 7435140 0 0.0
lighting-app debug+rpc (read only) 2318305 2318305 0 0.0
(read/write) 153504 153504 0 0.0
.bss 74720 74720 0 0.0
.data 2048 2048 0 0.0
.data.rel.ro 70984 70984 0 0.0
.dynamic 608 608 0 0.0
.got 4344 4344 0 0.0
.init 27 27 0 0.0
.init_array 792 792 0 0.0
.rodata 187400 187400 0 0.0
.text 1965202 1965202 0 0.0
lock-app debug (read only) 2258377 2258377 0 0.0
(read/write) 148632 148632 0 0.0
.bss 73440 73440 0 0.0
.data 1568 1568 0 0.0
.data.rel.ro 67912 67912 0 0.0
.dynamic 592 592 0 0.0
.got 4336 4336 0 0.0
.init 27 27 0 0.0
.init_array 752 752 0 0.0
.rodata 200904 200904 0 0.0
.text 1897202 1897202 0 0.0
ota-provider-app debug (read only) 2067441 2067441 0 0.0
(read/write) 141584 141584 0 0.0
.bss 73024 73024 0 0.0
.data 1768 1768 0 0.0
.data.rel.ro 60984 60984 0 0.0
.dynamic 608 608 0 0.0
.got 4504 4504 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 180184 180184 0 0.0
.text 1728210 1728210 0 0.0
ota-requestor-app debug (read only) 2096241 2096241 0 0.0
(read/write) 144424 144424 0 0.0
.bss 73728 73728 0 0.0
.data 1960 1960 0 0.0
.data.rel.ro 63080 63080 0 0.0
.dynamic 592 592 0 0.0
.got 4344 4344 0 0.0
.init 27 27 0 0.0
.init_array 672 672 0 0.0
.rodata 176256 176256 0 0.0
.text 1759474 1759474 0 0.0
shell debug (read only) 2573545 2573545 0 0.0
(read/write) 201680 201680 0 0.0
.bss 117096 117096 0 0.0
.data 1376 1376 0 0.0
.data.rel.ro 77448 77448 0 0.0
.dynamic 608 608 0 0.0
.got 4192 4192 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 223794 223794 0 0.0
.text 2190354 2190354 0 0.0
thermostat-no-ble arm64 (read only) 2366452 2366452 0 0.0
(read/write) 177425 177425 0 0.0
.bss 87921 87921 0 0.0
.data 1528 1528 0 0.0
.data.rel.ro 80176 80176 0 0.0
.dynamic 528 528 0 0.0
.got 4792 4792 0 0.0
.init 24 24 0 0.0
.init_array 376 376 0 0.0
.rodata 148164 148164 0 0.0
.text 1988704 1988704 0 0.0
tv-app debug (read only) 2876105 2876105 0 0.0
(read/write) 280176 280176 0 0.0
.bss 191048 191048 0 0.0
.data 4672 4672 0 0.0
.data.rel.ro 78200 78200 0 0.0
.dynamic 592 592 0 0.0
.got 4728 4728 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 222112 222112 0 0.0
.text 2471458 2471458 0 0.0
tv-casting-app debug (read only) 5437833 5437833 0 0.0
(read/write) 226160 226160 0 0.0
.bss 78632 78632 0 0.0
.data 2400 2400 0 0.0
.data.rel.ro 138912 138912 0 0.0
.dynamic 608 608 0 0.0
.got 4728 4728 0 0.0
.init 27 27 0 0.0
.init_array 864 864 0 0.0
.rodata 339808 339808 0 0.0
.text 4738434 4738434 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2424944 2424944 0 0.0
.bss 202620 202620 0 0.0
.data 5872 5872 0 0.0
.text 1387588 1387588 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1186495 1186495 0 0.0
bss 139276 139276 0 0.0
rodata 153584 153584 0 0.0
text 814728 814728 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1135227 1135227 0 0.0
bss 138508 138508 0 0.0
rodata 130148 130148 0 0.0
text 787728 787728 0 0.0
p6 all-clusters-app default (read/write) 2540816 2540816 0 0.0
.bss 137104 137104 0 0.0
.data 2808 2808 0 0.0
.text 1499080 1499080 0 0.0
all-clusters-minimal-app default (read/write) 2486760 2486760 0 0.0
.bss 136328 136328 0 0.0
.data 2744 2744 0 0.0
.text 1445024 1445024 0 0.0
light-app default (read/write) 2420712 2420712 0 0.0
.bss 129432 129432 0 0.0
.data 2600 2600 0 0.0
.text 1378976 1378976 0 0.0
lock-app default (read/write) 2438760 2438760 0 0.0
.bss 129248 129248 0 0.0
.data 2568 2568 0 0.0
.text 1397024 1397024 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 779708 779708 0 0.0
bss 70564 70564 0 0.0
noinit 40416 40416 0 0.0
text 551518 551518 0 0.0
lighting-app tlsr9518adk80d (read/write) 799736 799736 0 0.0
bss 70824 70824 0 0.0
noinit 40416 40416 0 0.0
text 568252 568252 0 0.0

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.

Fast tracking documentation updates

@cpagravel
Copy link
Contributor Author

Can we merge? There's no functional changes here.

@bzbarsky-apple
Copy link
Contributor

@cpagravel Please fix the restyled failures.

@github-actions
Copy link

github-actions bot commented Jun 9, 2022

PR #18980: Size comparison from d2e5909 to 55910c8

Increases (3 builds for cc13x2_26x2, telink)
platform target config section d2e5909 55910c8 change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read/write) 159368 159376 8 0.0
telink light-switch-app tlsr9518adk80d (read/write) 781664 781672 8 0.0
text 552544 552546 2 0.0
lighting-app tlsr9518adk80d (read/write) 801676 801684 8 0.0
text 569264 569266 2 0.0
Decreases (1 build for cc13x2_26x2)
platform target config section d2e5909 55910c8 change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read only) 683463 683455 -8 -0.0
.text 584796 584788 -8 -0.0
Full report (23 builds for cc13x2_26x2, cyw30739, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section d2e5909 55910c8 change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 689943 689943 0 0.0
(read/write) 161824 161824 0 0.0
.bss 74660 74660 0 0.0
.data 3392 3392 0 0.0
.rodata 112327 112327 0 0.0
.text 577304 577304 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 640599 640599 0 0.0
(read/write) 158132 158132 0 0.0
.bss 73884 73884 0 0.0
.data 3332 3332 0 0.0
.rodata 89367 89367 0 0.0
.text 550912 550912 0 0.0
lock-ftd LP_CC2652R7 (read only) 683463 683455 -8 -0.0
(read/write) 159368 159376 8 0.0
.bss 72612 72612 0 0.0
.data 3256 3256 0 0.0
.rodata 98183 98183 0 0.0
.text 584796 584788 -8 -0.0
lock-mtd LP_CC2652R7 (read only) 632879 632879 0 0.0
(read/write) 145720 145720 0 0.0
.bss 68348 68348 0 0.0
.data 3256 3256 0 0.0
.rodata 98071 98071 0 0.0
.text 534316 534316 0 0.0
pump-app LP_CC2652R7 (read only) 664923 664923 0 0.0
(read/write) 178820 178820 0 0.0
.bss 72756 72756 0 0.0
.data 3292 3292 0 0.0
.rodata 86363 86363 0 0.0
.text 578076 578076 0 0.0
pump-controller-app LP_CC2652R7 (read only) 655491 655491 0 0.0
(read/write) 188356 188356 0 0.0
.bss 72860 72860 0 0.0
.data 3252 3252 0 0.0
.rodata 84251 84251 0 0.0
.text 570756 570756 0 0.0
shell LP_CC2652R7 (read only) 682230 682230 0 0.0
(read/write) 165032 165032 0 0.0
.bss 76956 76956 0 0.0
.data 3396 3396 0 0.0
.rodata 108926 108926 0 0.0
.text 572992 572992 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 602562 602562 0 0.0
.app_xip_area 461508 461508 0 0.0
.bss 84008 84008 0 0.0
.data 732 732 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 599646 599646 0 0.0
.app_xip_area 458456 458456 0 0.0
.bss 84176 84176 0 0.0
.data 700 700 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 599446 599446 0 0.0
.app_xip_area 459372 459372 0 0.0
.bss 83140 83140 0 0.0
.data 616 616 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
k32w light k32w061+release (read/write) 653784 653784 0 0.0
.bss 70044 70044 0 0.0
.data 2004 2004 0 0.0
.text 575936 575936 0 0.0
lock k32w061+release (read/write) 714876 714876 0 0.0
.bss 70484 70484 0 0.0
.data 1976 1976 0 0.0
.text 636616 636616 0 0.0
linux chip-tool-no-interactive-ipv6only arm64 (read only) 9555948 9555948 0 0.0
(read/write) 689553 689553 0 0.0
.bss 43681 43681 0 0.0
.data 1152 1152 0 0.0
.data.rel.ro 625888 625888 0 0.0
.dynamic 528 528 0 0.0
.got 15032 15032 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 468332 468332 0 0.0
.text 7512212 7512212 0 0.0
thermostat-no-ble arm64 (read only) 2544060 2544060 0 0.0
(read/write) 183057 183057 0 0.0
.bss 91409 91409 0 0.0
.data 1512 1512 0 0.0
.data.rel.ro 82120 82120 0 0.0
.dynamic 528 528 0 0.0
.got 4992 4992 0 0.0
.init 24 24 0 0.0
.init_array 400 400 0 0.0
.rodata 160164 160164 0 0.0
.text 2147040 2147040 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2429656 2429656 0 0.0
.bss 202692 202692 0 0.0
.data 5872 5872 0 0.0
.text 1392300 1392300 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1192019 1192019 0 0.0
bss 141362 141362 0 0.0
rodata 154644 154644 0 0.0
text 817092 817092 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1139759 1139759 0 0.0
bss 140579 140579 0 0.0
rodata 131136 131136 0 0.0
text 789164 789164 0 0.0
p6 all-clusters-app default (read/write) 2545112 2545112 0 0.0
.bss 137120 137120 0 0.0
.data 2808 2808 0 0.0
.text 1503376 1503376 0 0.0
all-clusters-minimal-app default (read/write) 2489920 2489920 0 0.0
.bss 136328 136328 0 0.0
.data 2752 2752 0 0.0
.text 1448184 1448184 0 0.0
light-app default (read/write) 2421360 2421360 0 0.0
.bss 129432 129432 0 0.0
.data 2600 2600 0 0.0
.text 1379624 1379624 0 0.0
lock-app default (read/write) 2441696 2441696 0 0.0
.bss 129256 129256 0 0.0
.data 2576 2576 0 0.0
.text 1399960 1399960 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 781664 781672 8 0.0
bss 70636 70636 0 0.0
noinit 40416 40416 0 0.0
text 552544 552546 2 0.0
lighting-app tlsr9518adk80d (read/write) 801676 801684 8 0.0
bss 70888 70888 0 0.0
noinit 40416 40416 0 0.0
text 569264 569266 2 0.0

@cpagravel
Copy link
Contributor Author

@cpagravel Please fix the restyled failures.

I missed that, thanks.

@andy31415 andy31415 merged commit ba1b65d into project-chip:master Jun 10, 2022
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.

4 participants