Skip to content

Sync eng/common directory with azure-sdk-tools for PR 1564#752

Merged
weshaggard merged 1 commit intomasterfrom
sync-eng/common-StopFilteringByCIYml-1564
Apr 20, 2021
Merged

Sync eng/common directory with azure-sdk-tools for PR 1564#752
weshaggard merged 1 commit intomasterfrom
sync-eng/common-StopFilteringByCIYml-1564

Conversation

@azure-sdk
Copy link
Collaborator

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

- This removes the ci.yml filtering which means all projects can be discovered
- This eliminates the need for the yml parsing module for most common scenarios
- This does introduce some potential issues with duplicate package names for
  languages like java. The only known case currently is azure-storage-blobs and
  if there is ever a need to explicitly pick the correct one someone can pass
  in the ServiceDirectory filter similar to "storage/azure*" for new and "storage/microsoft*"
  for the older sdk. By default azure usually comes first so the new one gets discovered
  which is likely all we need for now. If this turns out not to be true we might
  need another way to disambiguate.
@azure-sdk azure-sdk added Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system. labels Apr 20, 2021
@weshaggard
Copy link
Member

Fix broken link with PR #753

@weshaggard
Copy link
Member

/check-enforcer override

@weshaggard weshaggard merged commit 5168343 into master Apr 20, 2021
@weshaggard weshaggard deleted the sync-eng/common-StopFilteringByCIYml-1564 branch April 20, 2021 21:59
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