-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Usermod libs matrix #4592
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
Usermod libs matrix #4592
Changes from all commits
27d3420
0e7d5dd
04c7eac
b3af04d
b1b2eea
8d4c911
5f19608
7d48bba
74672e2
99108f9
199529a
3a31d5d
adb9b77
7852ff5
b77881f
fbb7ef7
6c4d049
19ba257
92db9e0
f1c88bc
7835550
65a79d4
b0dd969
a6f5080
e97723d
1c4141a
7cc5c87
b187f94
5163fbf
a2e9e2b
b1ed99d
0ba0587
9099b13
db55fec
e227d01
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 | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,71 @@ | ||||||||||
| name: Usermod CI | ||||||||||
|
|
||||||||||
| on: | ||||||||||
| push: | ||||||||||
| paths: | ||||||||||
| - usermods/** | ||||||||||
| - .github/workflows/usermods.yml | ||||||||||
|
|
||||||||||
| jobs: | ||||||||||
|
|
||||||||||
| get_usermod_envs: | ||||||||||
| name: Gather Usermods | ||||||||||
| runs-on: ubuntu-latest | ||||||||||
| steps: | ||||||||||
| - uses: actions/checkout@v4 | ||||||||||
| - uses: actions/setup-python@v5 | ||||||||||
|
Comment on lines
+15
to
+16
Contributor
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. Fix YAML indentation under Apply this diff: - - uses: actions/checkout@v4
- - uses: actions/setup-python@v5
+ - uses: actions/checkout@v4
+ - uses: actions/setup-python@v5📝 Committable suggestion
Suggested change
🧰 Tools🪛 YAMLlint (1.37.1)[warning] 15-15: wrong indentation: expected 6 but found 4 (indentation) 🤖 Prompt for AI Agents |
||||||||||
| 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: | ||||||||||
| 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 | ||||||||||
|
Comment on lines
+40
to
+41
Contributor
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. Fix YAML indentation under Apply this diff: - - uses: actions/checkout@v4
- - name: Set up Node.js
+ - uses: actions/checkout@v4
+ - name: Set up Node.js📝 Committable suggestion
Suggested change
🧰 Tools🪛 YAMLlint (1.37.1)[warning] 40-40: wrong indentation: expected 6 but found 4 (indentation) 🤖 Prompt for AI Agents |
||||||||||
| 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' | ||||||||||
|
Contributor
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. Correct indentation for Python setup block. Apply this diff: - python-version: '3.12'
- cache: 'pip'
+ python-version: '3.12'
+ cache: 'pip'📝 Committable suggestion
Suggested change
🧰 Tools🪛 YAMLlint (1.37.1)[warning] 59-59: wrong indentation: expected 8 but found 10 (indentation) 🤖 Prompt for AI Agents |
||||||||||
| 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 }} | ||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,31 @@ | ||
| [platformio] | ||
| default_envs = usermods_esp32, usermods_esp32c3, usermods_esp32s2, usermods_esp32s3 | ||
|
|
||
| [env:usermods_esp32] | ||
| extends = env:esp32dev_V4 | ||
| custom_usermods = ${usermods.custom_usermods} | ||
| board_build.partitions = ${esp32.extreme_partitions} ; We're gonna need a bigger boat | ||
|
|
||
|
|
||
| [env:usermods_esp32c3] | ||
| extends = env:esp32c3dev | ||
| board = esp32-c3-devkitm-1 | ||
| custom_usermods = ${usermods.custom_usermods} | ||
| board_build.partitions = ${esp32.extreme_partitions} ; We're gonna need a bigger boat | ||
|
|
||
|
|
||
| [env:usermods_esp32s2] | ||
| extends = env:lolin_s2_mini | ||
| custom_usermods = ${usermods.custom_usermods} | ||
| board_build.partitions = ${esp32.extreme_partitions} ; We're gonna need a bigger boat | ||
|
|
||
|
|
||
| [env:usermods_esp32s3] | ||
| extends = env:esp32s3dev_16MB_opi | ||
| custom_usermods = ${usermods.custom_usermods} | ||
| board_build.partitions = ${esp32.extreme_partitions} ; We're gonna need a bigger boat | ||
|
|
||
|
|
||
|
|
||
| [usermods] | ||
| # Added in CI |
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.
Remove trailing whitespace – YAML-lint treats it as an error
Also applies to: 71-71
🧰 Tools
🪛 YAMLlint (1.37.1)
[error] 8-8: trailing spaces
(trailing-spaces)
🤖 Prompt for AI Agents