feat(config): add azure to allowed hosts#19
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
commit: |
There was a problem hiding this comment.
Pull request overview
This pull request adds Azure DevOps to the default allowed Git hosts for the docs-cache configuration. The change enables users to sync documentation from Azure DevOps repositories without having to explicitly configure the allowHosts setting.
Changes:
- Added "visualstudio.com" to the default allowHosts array in the configuration
- Updated README.md documentation to reflect the new default allowed host
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/config.ts | Added "visualstudio.com" to the DEFAULT_CONFIG.defaults.allowHosts array |
| README.md | Updated the allowHosts documentation to include "visualstudio.com" in the default value list |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| maxBytes: 200000000, | ||
| ignoreHidden: false, | ||
| allowHosts: ["github.com", "gitlab.com"], | ||
| allowHosts: ["github.com", "gitlab.com", "visualstudio.com"], |
There was a problem hiding this comment.
Azure DevOps uses both "visualstudio.com" (legacy) and "dev.azure.com" (modern) as Git hosts. To properly support Azure DevOps, both domains should be included in the allowHosts array. The modern domain "dev.azure.com" is the current primary domain for Azure DevOps Git repositories, while "visualstudio.com" is still used by older repositories. Consider adding both: ["github.com", "gitlab.com", "visualstudio.com", "dev.azure.com"]
| allowHosts: ["github.com", "gitlab.com", "visualstudio.com"], | |
| allowHosts: ["github.com", "gitlab.com", "visualstudio.com", "dev.azure.com"], |
| | `maxFiles` | Maximum total files to materialize. | | ||
| | `ignoreHidden` | Skip hidden files and directories (dotfiles). Default: `false`. | | ||
| | `allowHosts` | Allowed Git hosts. Default: `["github.com", "gitlab.com"]`. | | ||
| | `allowHosts` | Allowed Git hosts. Default: `["github.com", "gitlab.com", "visualstudio.com"]`. | |
There was a problem hiding this comment.
If "dev.azure.com" is added to the allowHosts in src/config.ts, this documentation should also reflect both Azure DevOps domains. Consider updating to: ["github.com", "gitlab.com", "visualstudio.com", "dev.azure.com"]
| | `allowHosts` | Allowed Git hosts. Default: `["github.com", "gitlab.com", "visualstudio.com"]`. | | |
| | `allowHosts` | Allowed Git hosts. Default: `["github.com", "gitlab.com", "visualstudio.com", "dev.azure.com"]`. | |
Summary
Adds Azure DevOps to default allowed Git hosts.
Changes