Closed MarkoSagadin closed 1 year ago
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
@MarkoSagadin I have made some edits, fixes and suggestions.
I also recommend you use east
and west
always in lowercase.
This was mentioned by @TjazVracko: https://developer.nordicsemi.com/nRF_Connect_SDK/doc/latest/zephyr/hardware/porting/board_porting.html#multiple-board-revisions
East should deduct a list of board revisions from cmake file that has VALID_REVISION
argument.
I also recommend you use east and west always in lowercase.
Makes sense, I will change that.
@MarkoSagadin
confs
and not conf
?child_image/mcuboot.conf
and child_image/spm.conf
Document looks good. For the begginer devs I would just add links to NCS / Zephyr docs about using Kconfig files and such.
- why is the folder named confs and not conf ?
- How will you handle child_image/mcuboot.conf and child_image/spm.conf
- Think about the case when mcuboot should be configured differently for the debug build than the release build. Will this be supported?
confs
into conf
child_image
folderDocument looks good. For the begginer devs I would just add links to NCS / Zephyr docs about using Kconfig files and such.
Will add!
@vid553 @TjazVracko Check again latest changes
Description
This PR adds a document that describes how East can be configured to specify build types and release procedures.
I plan to write a general
help
command so this document will be available from the command-line, but for now it will be just in thedocs
folder.Areas of interest
Read the file in the
docs
folder, I want to know if:After-review steps