Skip to content

Sync eng/common directory with azure-sdk-tools for PR 5427#20006

Merged
archerzz merged 1 commit intomainfrom
sync-eng/common-eng/cadl/fix-repo-url-match-5427
Feb 14, 2023
Merged

Sync eng/common directory with azure-sdk-tools for PR 5427#20006
archerzz merged 1 commit intomainfrom
sync-eng/common-eng/cadl/fix-repo-url-match-5427

Conversation

@azure-sdk
Copy link
Copy Markdown
Collaborator

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#5427 See eng/common workflow

`[.git]` is a char set which means the url should ends with either `.`, or `g`, or `i`, or `t`. So if the url doesn't end with `.git` then there is no match which results in error.

The fix is to remove `[.git]` char set. We don't need `(.git)*` either, because `\S+` will suppress it.
@azure-sdk azure-sdk requested a review from archerzz February 14, 2023 02:18
@azure-sdk azure-sdk added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Feb 14, 2023
@archerzz archerzz merged commit ea7cedd into main Feb 14, 2023
@archerzz archerzz deleted the sync-eng/common-eng/cadl/fix-repo-url-match-5427 branch February 14, 2023 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants