Skip to content

Commit 0bc86e9

Browse files
authored
Add memory statistics PR check (#94)
* Update for memory estimator changes * Add memory statistics PR check * Remove old uncrustify.cfg
1 parent 2899a1b commit 0bc86e9

File tree

5 files changed

+28
-170
lines changed

5 files changed

+28
-170
lines changed

.github/CONTRIBUTING.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ To send us a pull request, please:
3232
1. Fork the repository.
3333
1. Modify the source; please focus on the specific change you are contributing. If you also reformat all the code, it will be hard for us to focus on your change.
3434
1. Ensure that your contributions conform to the [style guide](https://docs.aws.amazon.com/embedded-csdk/202011.00/lib-ref/docs/doxygen/output/html/guide_developer_styleguide.html).
35+
1. Format your code with uncrustify, using the config available in [FreeRTOS/CI-CD-Github-Actions](https://github.com/FreeRTOS/CI-CD-Github-Actions/blob/main/formatting/uncrustify.cfg).
3536
1. Ensure local tests pass.
3637
1. Commit to your fork using clear commit messages.
3738
1. Send us a pull request, answering any default questions in the pull request interface.
Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
{
2+
"lib_name": "coreHTTP",
3+
"src": [
4+
"source/core_http_client.c",
5+
"source/dependency/3rdparty/http_parser/http_parser.c"
6+
],
7+
"include": [
8+
"source/include",
9+
"source/interface",
10+
"source/dependency/3rdparty/http_parser"
11+
],
12+
"compiler_flags": [
13+
"HTTP_DO_NOT_USE_CUSTOM_CONFIG"
14+
]
15+
}

.github/workflows/ci.yml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -109,3 +109,14 @@ jobs:
109109
run: |
110110
git-secrets --register-aws
111111
git-secrets --scan
112+
memory_statistics:
113+
runs-on: ubuntu-latest
114+
steps:
115+
- uses: actions/checkout@v2
116+
with:
117+
submodules: 'recursive'
118+
- name: Measure sizes
119+
uses: FreeRTOS/CI-CD-Github-Actions/memory_statistics@main
120+
with:
121+
config: .github/memory_statistics_config.json
122+
check_against: docs/doxygen/include/size_table.html

.github/workflows/memory_statistics.yml

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -14,16 +14,7 @@ jobs:
1414
- name: Measure sizes
1515
uses: FreeRTOS/CI-CD-Github-Actions/memory_statistics@main
1616
with:
17-
lib_name: coreHTTP
18-
src: |
19-
source/core_http_client.c
20-
source/dependency/3rdparty/http_parser/http_parser.c
21-
include: |
22-
source/include
23-
source/interface
24-
source/dependency/3rdparty/http_parser
25-
compiler_flags: |
26-
HTTP_DO_NOT_USE_CUSTOM_CONFIG
17+
config: .github/memory_statistics_config.json
2718
- name: Upload table
2819
uses: actions/upload-artifact@v2
2920
with:

tools/uncrustify.cfg

Lines changed: 0 additions & 160 deletions
This file was deleted.

0 commit comments

Comments
 (0)