-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
ESP-IDF V5 #4838
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
base: main
Are you sure you want to change the base?
ESP-IDF V5 #4838
Changes from all commits
965e794
ba2b182
07ab6aa
1de8c2e
2c0259f
03a9d9e
bc19133
e4815d1
81c9225
ec22c50
68dff2d
7daada1
446c043
d4bafec
2f8882e
5205110
906d456
41878f0
44a483f
7a0325f
86679ed
c5e2ec7
93821ef
8cad34e
ce84a57
2af23fc
51e1f4f
0653854
36eaca6
7c8b8fd
d632cf8
b51e80f
9f3e53a
9743775
79e2128
ecfcd0d
7e46616
a552094
2c7923e
83b541e
7192065
e891c5c
2245e0e
5ef41e7
8ea41cf
0b27bc3
5d54597
29c7b50
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,74 +1,74 @@ | ||
| name: Usermod CI | ||
|
|
||
| on: | ||
| push: | ||
| paths: | ||
| - usermods/** | ||
| - .github/workflows/usermods.yml | ||
| pull_request: | ||
| paths: | ||
| - usermods/** | ||
|
|
||
| jobs: | ||
|
|
||
| get_usermod_envs: | ||
| # Only run for pull requests from forks (not from branches within wled/WLED) | ||
| if: github.event.pull_request.head.repo.full_name != github.repository | ||
| name: Gather Usermods | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-python@v5 | ||
| with: | ||
| python-version: '3.12' | ||
| cache: 'pip' | ||
| - name: Install PlatformIO | ||
| run: pip install -r requirements.txt | ||
| - name: Get default environments | ||
| id: envs | ||
| run: | | ||
| echo "usermods=$(find usermods/ -name library.json | xargs dirname | xargs -n 1 basename | jq -R | grep -v PWM_fan | grep -v BME68X_v2| grep -v pixels_dice_tray | jq --slurp -c)" >> $GITHUB_OUTPUT | ||
| outputs: | ||
| usermods: ${{ steps.envs.outputs.usermods }} | ||
|
|
||
|
|
||
| build: | ||
Check warningCode scanning / CodeQL Workflow does not contain permissions Medium
Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}
|
||
| # Only run for pull requests from forks (not from branches within wled/WLED) | ||
| if: github.event.pull_request.head.repo.full_name != github.repository | ||
| name: Build Enviornments | ||
| runs-on: ubuntu-latest | ||
| needs: get_usermod_envs | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| usermod: ${{ fromJSON(needs.get_usermod_envs.outputs.usermods) }} | ||
| environment: [usermods_esp32, usermods_esp32c3, usermods_esp32s2, usermods_esp32s3] | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - name: Set up Node.js | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version-file: '.nvmrc' | ||
| cache: 'npm' | ||
| - run: npm ci | ||
| - name: Cache PlatformIO | ||
| uses: actions/cache@v4 | ||
| with: | ||
| path: | | ||
| ~/.platformio/.cache | ||
| ~/.buildcache | ||
| build_output | ||
| key: pio-${{ runner.os }}-${{ matrix.environment }}-${{ hashFiles('platformio.ini', 'pio-scripts/output_bins.py') }}-${{ hashFiles('wled00/**', 'usermods/**') }} | ||
| restore-keys: pio-${{ runner.os }}-${{ matrix.environment }}-${{ hashFiles('platformio.ini', 'pio-scripts/output_bins.py') }}- | ||
| - name: Set up Python | ||
| uses: actions/setup-python@v5 | ||
| with: | ||
| python-version: '3.12' | ||
| cache: 'pip' | ||
| - name: Install PlatformIO | ||
| run: pip install -r requirements.txt | ||
| - name: Add usermods environment | ||
| run: | | ||
| cp -v usermods/platformio_override.usermods.ini platformio_override.ini | ||
| echo >> platformio_override.ini | ||
| echo "custom_usermods = ${{ matrix.usermod }}" >> platformio_override.ini | ||
| cat platformio_override.ini | ||
|
|
||
| - name: Build firmware | ||
| run: pio run -e ${{ matrix.environment }} | ||
Check warningCode scanning / CodeQL Workflow does not contain permissions Medium
Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,4 @@ | ||
| #if false | ||
| /*------------------------------------------------------------------------- | ||
| NeoPixel driver for ESP32 RMTs using High-priority Interrupt | ||
|
|
||
|
|
@@ -467,3 +468,5 @@ typedef NeoEsp32RmtHI7Ws2805InvertedMethod NeoEsp32RmtHI7Ws2814InvertedMethod; | |
| #endif // !defined(CONFIG_IDF_TARGET_ESP32C3) | ||
|
|
||
| #endif | ||
|
|
||
| #endif // NEOE_SP32_RMT_HI_METHODS_H | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. TODO: remove once this has been updated for newer ESP-IDF |
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,5 @@ | ||
| #if false | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. TODO: remove once this has been updated for newer ESP-IDF |
||
|
|
||
| /*------------------------------------------------------------------------- | ||
| NeoPixel library helper functions for Esp32. | ||
|
|
||
|
|
@@ -504,4 +506,6 @@ esp_err_t NeoEsp32RmtHiMethodDriver::WaitForTxDone(rmt_channel_t channel, TickTy | |
| return rv; | ||
| } | ||
|
|
||
| #endif | ||
| #endif | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. TODO: remove once this has been updated for newer ESP-IDF |
||
|
|
||
| #endif // ARDUINO_ARCH_ESP32 | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: remove once this has been updated for newer ESP-IDF