Repository navigation
[azure-iot] Align wait and migration lint exclusions - #10441
Merged
JaysonTaiMicrosoft merged 2 commits intoOct 8, 2026
Merged
JaysonTaiMicrosoft merged 2 commits into
JaysonTaiMicrosoft merged 2 commits into
Conversation
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 22:49
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The narrowly scoped configuration changes match the stated intent and have no identified blocking issues.
Review effort: Balanced
Findings: None
What changed in this PR
Aligns azure-iot lint exclusions with existing command behavior without changing commands or src/index.json.
Changes:
- Adds six
require_wait_command_if_no_waitexclusions. - Corrects two
rule_exclusionskeys for resource-group parameters oniot hub state migrate.
| File | Description |
|---|---|
| linter_exclusions.yml | Adds wait-rule exclusions and fixes migration parameter exclusion keys. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Collaborator
|
azure-iot |
Align the existing IoT exemption for listing topic groups within an ARM-identified Hub. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
JaysonTaiMicrosoft
approved these changes
Oct 8, 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.
Add six existing azure-iot
require_wait_command_if_no_waitexclusions:dt endpoint create: already has parent-leveldt endpoint wait.iot dps linked-hub,iot dps policy, andiot hub policy: mutations use parent-resource LROs, not independently provisioned child resources.iot adr ns report: generation uses an LRO; the latest report is not operation-completion status.iot hub: retain the existing wait-by-default /--no-waitinterface without adding a standalone waiter.Add the existing
no_ids_for_list_commandsexclusion foriot hub topic-group list:--idsidentifies the parent IoT Hub whose topic groups are listed.Correct two
rule exclusionskeys torule_exclusionsfor the origin/destination resource-group options oniot hub state migrate.Only lint configuration changes; no command or
src/index.jsonchanges.