Skip to content

feat: proper exception on invalid soc statement - #406

Merged
hfudev merged 1 commit into
espressif:mainfrom
horw:feat/raise-error-on-wrong-soc-statement
Mar 2, 2026
Merged

feat: proper exception on invalid soc statement#406
hfudev merged 1 commit into
espressif:mainfrom
horw:feat/raise-error-on-wrong-soc-statement

Conversation

@horw

@horw horw commented Mar 2, 2026

Copy link
Copy Markdown
Collaborator

Description

This fix will help make the error more explicit, since it’s possible that the user inputs a SOC statement, which doesn’t work.

https://github.com/espressif/idf-extra-components/actions/runs/22336912154/job/64632505958?pr=681

Related

Testing


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@horw
horw requested a review from hfudev March 2, 2026 05:53
@github-actions

github-actions Bot commented Mar 2, 2026

Copy link
Copy Markdown
Title Coverage Tests Skipped Failures Errors Time
Qemu Coverage 7 0 💤 0 ❌ 0 🔥 25.43s ⏱️
3.14 X64 Coverage 103 14 💤 2 ❌ 0 🔥 13m 46s ⏱️
3.10 ARM64 Coverage 103 16 💤 3 ❌ 0 🔥 10m 45s ⏱️

@hfudev
hfudev merged commit 60e9456 into espressif:main Mar 2, 2026
4 of 6 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.

2 participants