Repository navigation
[azure-iot] Exempt digital-twin update authentication default from lint - #10433
Merged
Ethan Yang (necusjz) merged 1 commit intoOct 6, 2026
Conversation
Match existing Hub update-command exemptions for auth_type_dataplane. This default selects request authentication rather than updating a resource property. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Hangyi (HangyiWang)
requested a review
from ZelinWang (wangzelin007)
as a code owner
October 6, 2026 03:51
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The narrowly scoped exemption matches established neighboring entries and the upstream extension configuration.
Review effort: Balanced
Findings: None
What changed in this PR
Adds the missing parameter-scoped lint exemption for Azure IoT digital-twin authentication defaults.
Changes:
- Exempts
auth_type_dataplaneoniot hub digital-twin updatefrom the update-default rule. - Aligns lint behavior with related IoT Hub commands and the IoT extension repository.
| File | Description |
|---|---|
linter_exclusions.yml |
Adds the targeted digital-twin lint exemption. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Ethan Yang (necusjz)
approved these changes
Oct 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add a parameter-scoped
no_parameter_defaults_for_update_commandsexemption foriot hub digital-twin update/auth_type_dataplane.The default
keyselects request authentication; it does not update a digital-twin property. This follows the existing exemptions foriot hub device-twin update,device-identity update,module-twin update, and other Hub update commands.The same exemption already exists in the IoT extension repository, where release lint applies it. The index repository was missing it, causing the external-wheel linter to report a HIGH-severity finding for the published preview.
Scope
linter_exclusions.ymlchanges: five added lines.Related: #10432. The
1.0.0b2index proposal remains a separate, index-only PR.Validation
azure-iot1.0.0b2wheel in an isolated extension directory and ran the actualazdev 0.2.13rule against its command table.device-twin updateexemption.git diff --checkpassed.