fix(azure): use Azure icon for Azure DevOps connector and remove AzureDevOpsIcon#5118
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview The connector utils test icon mock is updated so imports still resolve after the rename. Reviewed by Cursor Bugbot for commit 7802934. Configure here. |
Greptile SummaryThis PR replaces the dedicated
Confidence Score: 5/5Safe to merge — purely a cosmetic icon swap with no logic, API, or data changes. All four consumers of AzureDevOpsIcon in the sim app are updated consistently, the test mock is kept in sync, and the deleted component has no remaining references in apps/sim. The only loose end is the orphaned AzureDevOpsIcon definition in apps/docs/components/icons.tsx, which is a pre-existing condition unaffected by this PR. No files require special attention. apps/docs/components/icons.tsx retains a now-unused AzureDevOpsIcon definition, but it is outside this PR's scope. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[AzureIcon\napps/sim/components/icons.tsx] --> B[azure-devops/meta.ts\nConnector]
A --> C[azure_devops/build_failed.ts\nTrigger]
A --> D[azure_devops/webhook.ts\nTrigger]
A --> E[azure_devops/work_item_created.ts\nTrigger]
F["~~AzureDevOpsIcon~~ (removed)"] -. was used by .-> B
F -. was used by .-> C
F -. was used by .-> D
F -. was used by .-> E
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
A[AzureIcon\napps/sim/components/icons.tsx] --> B[azure-devops/meta.ts\nConnector]
A --> C[azure_devops/build_failed.ts\nTrigger]
A --> D[azure_devops/webhook.ts\nTrigger]
A --> E[azure_devops/work_item_created.ts\nTrigger]
F["~~AzureDevOpsIcon~~ (removed)"] -. was used by .-> B
F -. was used by .-> C
F -. was used by .-> D
F -. was used by .-> E
Reviews (1): Last reviewed commit: "fix(azure): replace Azure DevOps icon wi..." | Re-trigger Greptile |
Summary
AzureDevOpsIconentirely; repointed the three Azure DevOps triggers toAzureIconconnectors/utils.test.tsType of Change
Testing
Tested manually
Checklist