Feature: make-pretty enabled for all languages / Updates: Github yml build files #169
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…lds in build.yml; bring checkout before setup-go to avoid warnings.
…h Lint requiring a node-build.
EskoDijk
added a commit
that referenced
this pull request
Sep 24, 2024
…les and make-pretty applied (#169) This updates the `make-pretty` script to support all used languages in the project. It's based on the same script in the `openthread` repo. Also make-pretty is applied on all sources. Additionally, Github yml files are updated to better comply with recommended steps order and to bump setup-go to v5. Some steps are renamed to provide more clarity what it does. And the 'build' test is now split in the Go build part and the OT nodes build part. These 2 parts each require a separate matrix (since the nodes build doesn't involve Go, and the Go build doesn't involve C/C++ ) * [ot-rfsim] make-pretty clang-tidy * [ot-rfsim] make-pretty clang-format * [all] make-pretty markdown .md * [ot-rfsim] make-pretty shell * [pylibs][script] make-pretty python (excluding generates files in proto/) * [.github] Updated Github yml workflow names; split OTNS and Nodes builds in build.yml; bring checkout before setup-go to avoid warnings. I.e. apply standard order as recommended. * [script] moved cmake and ninja to requires packages -> needed now with Lint requiring a node-build.
EskoDijk
added a commit
that referenced
this pull request
Sep 24, 2024
…les and make-pretty applied (#169) This updates the `make-pretty` script to support all used languages in the project. It's based on the same script in the `openthread` repo. Also make-pretty is applied on all sources. Additionally, Github yml files are updated to better comply with recommended steps order and to bump setup-go to v5. Some steps are renamed to provide more clarity what it does. And the 'build' test is now split in the Go build part and the OT nodes build part. These 2 parts each require a separate matrix (since the nodes build doesn't involve Go, and the Go build doesn't involve C/C++ ) * [ot-rfsim] make-pretty clang-tidy * [ot-rfsim] make-pretty clang-format * [all] make-pretty markdown .md * [ot-rfsim] make-pretty shell * [pylibs][script] make-pretty python (excluding generates files in proto/) * [.github] Updated Github yml workflow names; split OTNS and Nodes builds in build.yml; bring checkout before setup-go to avoid warnings. I.e. apply standard order as recommended. * [script] moved cmake and ninja to requires packages -> needed now with Lint requiring a node-build.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This enables 'make-pretty' for all included languages; and applies make-pretty to all sources.
Also, updates Github yml build files as indicated in commits.