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

CI-CD Updates and Repository PR Checks #68

Merged
merged 139 commits into from
Sep 6, 2023
Merged

CI-CD Updates and Repository PR Checks #68

merged 139 commits into from
Sep 6, 2023

Conversation

Skptak
Copy link
Member

@Skptak Skptak commented Aug 14, 2023

Minor updates to some of the CI-CD files to remove trailing whitespace, and fix formatting issues
Use checkout@v3 in all places instead of v2
Fixes to the README for the repo
Add in a new test that runs on PRs into this repo so that potential changes are tested against known consumers of this repository

Skptak and others added 30 commits August 5, 2023 14:59
    * Swap to using clang-format instead of uncrustify
    * Use a rust spell checker instead of the current one
    * Use echo groups to make logs easier to parse
    * Add a PR Check that runs against all repos
    * Idea of common config templates for repos that use this
    ---------
@Skptak Skptak force-pushed the main branch 3 times, most recently from acd8fba to efb70b2 Compare September 2, 2023 22:04
…o the spelling action to make it work when run from alternate directories
@Skptak Skptak changed the title CI-CD-Overhaul CI-CD Updates and Repository PR Checks Sep 5, 2023
@Skptak Skptak force-pushed the main branch 3 times, most recently from b30c94d to 4f3d626 Compare September 6, 2023 20:31
bradleysmith23
bradleysmith23 previously approved these changes Sep 6, 2023
def pacakge_writer(output, packageName: str, version: str, url: str, license: str, ver_code: str, file_analyzed=True,

def pacakge_writer(output, packageName: str, version: str, url: str, license: str, ver_code: str, file_analyzed=True,
Copy link

Choose a reason for hiding this comment

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

NIT: Spelling -- package_writer

A-Zaba
A-Zaba previously approved these changes Sep 6, 2023
@Skptak Skptak merged commit 5f0b1e7 into FreeRTOS:main Sep 6, 2023
50 checks passed
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.

5 participants